Skip to content

Conversation

@jhrcek
Copy link
Contributor

@jhrcek jhrcek commented Sep 20, 2025

It seems like github actions CI hasn't been working for some time.
This is likely because the generated config uses ubuntu-20 images, which have been recently decomissioned (see https://github.blog/changelog/2025-01-15-github-actions-ubuntu-20-runner-image-brownout-dates-and-other-breaking-changes/). Regenerating config with more recent version of haskell-ci should fix this (uses ubuntu 24 by default).

Source-Repository head
type: git
location: git://github.com/jtdaugherty/brick.git
location: http://github.com/jtdaugherty/brick
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixing cabal check warning (executed as part of generated haskell-ci config)

Error: [git-protocol] Cloning over git:// might lead to an arbitrary code
execution vulnerability. Furthermore, popular forges like GitHub do not
support it. Use https:// or ssh:// instead.

echo " ghc-options: -Werror=missing-methods" >> cabal.project
echo " ghc-options: -Werror=missing-methods -Werror=missing-fields" >> cabal.project
if [ $((HCNUMVER >= 90400)) -ne 0 ] ; then echo "package brick" >> cabal.project ; fi
if [ $((HCNUMVER >= 90400)) -ne 0 ] ; then echo " ghc-options: -Werror=unused-packages" >> cabal.project ; fi
Copy link
Contributor Author

@jhrcek jhrcek Sep 20, 2025

Choose a reason for hiding this comment

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

Recent version of haskell-ci adds this warning, which causes build error if there are unused dependencies in cabal file - for that reason removed all the unused dependencies to fix build failures caused by this warning.

@jtdaugherty jtdaugherty merged commit da64017 into jtdaugherty:master Sep 20, 2025
8 checks passed
@jtdaugherty
Copy link
Owner

Thank you for your work on this!

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