Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: improve longest_palindromic_subsequence.cpp #2467

Merged
merged 1 commit into from
May 20, 2023
Merged

Conversation

Panquesito7
Copy link
Member

Description of Change

  • Improve longest_palindromic_subsequence.cpp.
    • Add missing parameter/return statement documentation.
    • Add namespace for consistency.
    • Add a link that explains the algorithm.

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:

@Panquesito7 Panquesito7 added the enhancement New feature or request label May 18, 2023
@github-actions github-actions bot added the approved Approved; waiting for merge label May 18, 2023
@Panquesito7 Panquesito7 merged commit 4f4585d into master May 20, 2023
@Panquesito7 Panquesito7 deleted the improve_lps branch May 20, 2023 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Approved; waiting for merge enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants