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

fix retest title change #539

Merged
merged 2 commits into from
Jun 6, 2024
Merged

fix retest title change #539

merged 2 commits into from
Jun 6, 2024

Conversation

kwk
Copy link
Collaborator

@kwk kwk commented Jun 6, 2024

Before the /retest command internally fetched the correct information from the issue in which the /retest <chroot> comment was made. But it didn't use it and accidentally a property (github.issue_title) was used that internally used the current date 🤦 . I've changed the @property to become a regular function and now hopefully we should not see the title being renamed.

NOTE:
You could asked why we edit the title at all and that is a good question. In the past I've experimented with different title formats and I wanted the new format to always show the next a time the check would run.

Fix #538

@kwk kwk merged commit 0df206d into main Jun 6, 2024
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/retest automation should respect date in issue title it was being run in
2 participants