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

Column Badge Documentation #962

Merged
merged 1 commit into from
Mar 19, 2021
Merged

Column Badge Documentation #962

merged 1 commit into from
Mar 19, 2021

Conversation

alldoami
Copy link
Contributor

Summary of Changes
Added examples of how to use API for column badge

Documentation
Relevant to: amundsen-io/amundsenmetadatalibrary#273

@alldoami alldoami requested review from feng-tao and a team as code owners March 16, 2021 22:58
@alldoami
Copy link
Contributor Author

@feng-tao could you help me with merging non-related changes 😭

@feng-tao
Copy link
Member

@alldoami could you rebase your local branch with the upstream master? lots of commits have been merged already which just lack in your local fork.

@alldoami
Copy link
Contributor Author

@feng-tao i think I rebased correctly, but I need to sign now. I only want to sign the two commits that aren't signed but it goes back 78 in the instructions from DCO.

@dorianj
Copy link
Contributor

dorianj commented Mar 17, 2021

this rebase didn't work -- there are still other merge commits present.

@alldoami
Copy link
Contributor Author

@dorianj I do not see the files changed anymore though...

@feng-tao
Copy link
Member

there is a unknown file ( amundsen-kube-helm/templates/helm/charts/elasticsearch-1.32.5.tgz) added to the pr.

@feng-tao
Copy link
Member

@alldoami do you think you could open a new pr ? The long list of past commit concerns me a bit. Also you could sign the just one commit for the new pr. Sorry for the inconvenience.

@alldoami
Copy link
Contributor Author

alldoami commented Mar 17, 2021

@feng-tao fixed!

@alldoami alldoami closed this Mar 17, 2021
@alldoami alldoami reopened this Mar 17, 2021
Signed-off-by: adoami <adoami@chanzuckerberg.com>
@feng-tao feng-tao merged commit 07cdd44 into amundsen-io:master Mar 19, 2021
markgrover pushed a commit to markgrover/amundsen that referenced this pull request Apr 12, 2021
Signed-off-by: adoami <adoami@chanzuckerberg.com>
dorianj pushed a commit to dorianj/amundsen that referenced this pull request Apr 25, 2021
Signed-off-by: adoami <adoami@chanzuckerberg.com>
feng-tao pushed a commit that referenced this pull request May 7, 2021
Signed-off-by: adoami <adoami@chanzuckerberg.com>
prafullkotecha pushed a commit to prafullkotecha/amundsen that referenced this pull request May 19, 2021
Signed-off-by: adoami <adoami@chanzuckerberg.com>
Signed-off-by: GitHub <noreply@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants