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

amend typos in get_dashboard_tables #2222

Merged
merged 2 commits into from
Dec 19, 2023

Conversation

sihaotan4
Copy link
Contributor

Summary of changes

Corrected 3 typos in _get_dashboard_tables where "table" was misspelled.

How Has This Been Tested?

Tested internally on Neptune server. This change does not affect function logic and the function's return value in any way.

Documentation

No documentation added.

CheckList

  • PR title addresses the issue accurately and concisely
  • Updates Documentation and Docstrings
  • Adds tests
  • Adds instrumentation (logs, or UI events)

@sihaotan4 sihaotan4 requested a review from a team as a code owner December 18, 2023 11:25
@boring-cyborg boring-cyborg bot added the area:metadata From the metadata folder label Dec 18, 2023
Copy link

boring-cyborg bot commented Dec 18, 2023

Congratulations on your first Pull Request and welcome to Amundsen community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/amundsen-io/amundsen/blob/main/CONTRIBUTING.md)

@Golodhros
Copy link
Member

@sihaotan4 please check the details on the DCO task before we can review

@kristenarmes
Copy link
Contributor

@sihaotan4 can you bump the metadata version here to 3.12.3 so we can do a new release? then we can approve the changes, thanks!

Signed-off-by: sihaotan4 <[email protected]>
@kristenarmes kristenarmes merged commit ae5fb6d into amundsen-io:main Dec 19, 2023
8 checks passed
Copy link

boring-cyborg bot commented Dec 19, 2023

Awesome work, congrats on your first merged pull request!

@sihaotan4 sihaotan4 deleted the fix_table_typos branch December 19, 2023 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:metadata From the metadata folder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants