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

DM-47418: Communicate build result with GitHub #43

Draft
wants to merge 130 commits into
base: main
Choose a base branch
from

Conversation

aranabhat
Copy link

Adding github status posts from corresponding jenkins runs.

@aranabhat aranabhat requested a review from roceb January 21, 2025 11:36
@timj timj changed the title tickets/DM-47418 DM-47418: Communicate build result with GitHub Jan 21, 2025
@timj
Copy link
Member

timj commented Jan 21, 2025

@aranabhat consider using pre-commit to ensure that your code conforms with black/isort before you commit it (pre-commit install will do it -- you may need to also do a pre-commit autoupdate and commit the new pre commit config file since it's going to be out of date).

Just in case it's not clear, when it comes time to review you should probably redo the commit history from scratch.

more code cleanup

more code cleanup

cleanup
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.

2 participants