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.

Sign up for GitHub

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

Jump to bottom

Introduce 'reason' output #180

Merged
merged 5 commits into from Jan 16, 2022
Merged

Introduce 'reason' output #180

merged 5 commits into from Jan 16, 2022

Conversation

paescuj
Copy link
Collaborator

@paescuj paescuj commented Jan 16, 2022 •
edited

See #159

  • reason: The reason why the current run is considered skippable or unskippable.
    • For example: skip_after_successful_duplicate

To be merged after #179

This was referenced Jan 16, 2022
Closed
Merged
fkirc
fkirc approved these changes Jan 16, 2022
View reviewed changes
Copy link
Owner

@fkirc fkirc left a comment

Choose a reason for hiding this comment

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

Lgtm, thank you for splitting everything into small chunks and keeping docs updated 👍

@fkirc fkirc merged commit bda236d into fkirc:master Jan 16, 2022
@paescuj paescuj deleted the output-skip-reason branch January 16, 2022 21:15
@paescuj
Copy link
Collaborator Author

paescuj commented Jan 16, 2022

Lgtm, thank you for splitting everything into small chunks and keeping docs updated 👍

Thanks as well 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fkirc fkirc fkirc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@paescuj @fkirc