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

test(cli): Fix failing cli tests #2453

Merged
merged 2 commits into from
Sep 20, 2024
Merged

test(cli): Fix failing cli tests #2453

merged 2 commits into from
Sep 20, 2024

Conversation

sirtimid
Copy link
Contributor

@sirtimid sirtimid commented Sep 13, 2024

Closes: #2409
Refs: #2384

Description

This PR updates the readme to properly document the list cli command and fixes the failing tests

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

none

Compatibility Considerations

none

Upgrade Considerations

none

@sirtimid sirtimid marked this pull request as ready for review September 13, 2024 11:54
Copy link
Contributor

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, the incremental commits of this PR need to to be converted into conventional commits.

@sirtimid sirtimid changed the title test(cli): Fix failing tests and update Readme test(cli): Fix failing cli tests Sep 19, 2024
Copy link
Contributor

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kriskowal
Copy link
Member

@sirtimid Please rebase on origin/master, force push, then use the “create a merge commit” button. This will keep the history clean: parallel linked lists with occasional merge knots.

@kriskowal
Copy link
Member

@sirtimid Since you only have one significant commit, “squash merge” is also fine. The default commit message for the merge commit is your description above, which is also fine as writ.

@sirtimid sirtimid merged commit 7948a69 into master Sep 20, 2024
15 checks passed
@sirtimid sirtimid deleted the sirtimid/fix-cli-tests branch September 20, 2024 09:56
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.

docs(cli): update --as expectations
3 participants