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

chore: add info about the Matrix community chat channel #220

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

JohnVillalovos
Copy link
Collaborator

No description provided.

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/matrix branch 3 times, most recently from ac77249 to 2198dee Compare June 14, 2024 19:49
@StykMartin
Copy link
Contributor

StykMartin commented Jun 14, 2024

/packit build

@JohnVillalovos
Copy link
Collaborator Author

/packit build

2 similar comments
@JohnVillalovos
Copy link
Collaborator Author

/packit build

@JohnVillalovos
Copy link
Collaborator Author

/packit build

@JohnVillalovos
Copy link
Collaborator Author

@StykMartin : FYI @nforro is looking into why Packit appears to be getting stuck in updating the check results.

@nforro
Copy link

nforro commented Jun 24, 2024

@JohnVillalovos It turns out it is a known limitation of Packit, we have updated the docs to make it more clear. To avoid this, you have to set a different identifier option for each copr_build job; then the status check updates will be done independently for each of them and should be always up-to-date.
I've also noticed you are still using the metadata key in your Packit config; it has been deprecated long ago and you can safely drop it.

@JohnVillalovos
Copy link
Collaborator Author

@JohnVillalovos It turns out it is a known limitation of Packit, we have updated the docs to make it more clear. To avoid this, you have to set a different identifier option for each copr_build job; then the status check updates will be done independently for each of them and should be always up-to-date. I've also noticed you are still using the metadata key in your Packit config; it has been deprecated long ago and you can safely drop it.

Thanks @nforro I have created a PR to do what you suggested. #224

That PR passed CI. So hopefully that is the fix. I say hopefully because other PRs have passed before.

Thanks again.

@JohnVillalovos JohnVillalovos merged commit eb5c16c into python-3 Jun 24, 2024
12 checks passed
@JohnVillalovos JohnVillalovos deleted the jlvillal/matrix branch June 24, 2024 21:11
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