Fix CI: install libgirepository-2.0-dev required by newer PyGObject b… - #2341
Fix CI: install libgirepository-2.0-dev required by newer PyGObject b…#2341anhtumai wants to merge 1 commit into
Conversation
|
@Davidy22 this PR should unblock the CI pipeline: https://github.com/Guake/guake/actions/runs/28706200405/job/85132147751 , which fails with: ../meson.build:35:9: ERROR: Dependency 'girepository-2.0' is required but not found.Can you run this pipeline to test: https://github.com/Guake/guake/actions/runs/30938832362 |
|
Man the code rot is real, immediately runs into another error, at least you made the libgi error disappear. Sorry about the slow response, been busy. Do you want to dig through all the CI rot and make that into one PR or leave the rest to someone else? |
|
Since I cannot run the ci pipeline, I just throw out the fix and pray. I will find a way to simulate the CI environment and fix the next error |
|
I want to fix it properly |
|
You should just be able to enable it on your fork. Thanks for taking this on, been busy lately |
57bb41a to
fdd5d24
Compare
Why these pins exist
|
|
@Davidy22 the action on my PR has succeeded with this pin: https://github.com/anhtumai/guake/actions/runs/31902412202/job/95055080045 Can you rerun the main job to check? P/S: if possible, you can rerun |
…uilds and fix some pylint stylings
fdd5d24 to
09f6508
Compare
Davidy22
left a comment
There was a problem hiding this comment.
The run works again, so thank you for your contribution, I just noticed one thing that seems out of scope for this PR, and doesn't seem to be quite correct on its own merit.
On a side note I have been noticing an uptick in PRs lately, and the big hunk of text you dropped does look a bit like AI. Was there some article everyone read or something that I missed? I'm not gonna ban AI, but I'm still going to expect PRs to meet requirements and be correct
| # Stop the event here: otherwise VTE also forwards the click to the | ||
| # application running in the terminal, which acts on it a second time. | ||
| return True | ||
| elif event.button == 3 and matched_string: |
There was a problem hiding this comment.
Why change this? Doesn't have anything to do with CI
There was a problem hiding this comment.
this fails pylint test.
`no-else-return` check at terminal.py:366
So I need to change elif to if. Should be safe to do so.
There was a problem hiding this comment.
about the big chunk of text above
Yes, I did rely on AI to search for information , and the right version of package to pin, but I still reviewed all the changes and regenerated lock file myself, so AI cannot halluciate incorrect hash.
I wanted to document why these pins fixed CI issues, but got a bit lazy to write it myself, so used AI to do so.
There was a problem hiding this comment.
but yeah, I always reviewed the code , and mostly used AI for boring tasks like writing tests or documentation (but I also reviewed them afterward). I hope it is ok
…uilds
Please follow these steps before submitting a new Pull Request to Guake:
rebase on latest HEAD:
hack your change
to execute the code styling, checks and unit tests:
$ make style check reno-lint testdescribe your change in a slug file for automatic release note
generation, using:
and edit the created file in
releasenotes/notes/.You can see how
renoworks usingpipenv run reno --help.Please use a generic slug (eg, for translation update,
use
translation, for bugfix usebugfix,...)create new commit message
If your change is related to a GitHub issue, you can add a reference
using
#123where 123 is the ID of the issue.You can use
closes #123to have GitHub automatically close the issuewhen your contribution get merged
Semantic commit is supported (and recommended). Add one of the following
line in your commit messages: