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

Refactoring for bzlmod #1905

Merged
merged 5 commits into from
Jun 12, 2023
Merged

Refactoring for bzlmod #1905

merged 5 commits into from
Jun 12, 2023

Conversation

ylecornec
Copy link
Member

Depends on #1903

This PR contains some more refactoring in preparation for bzlmod.

  • 84be611 adds a rules_haskell_dependencies_bzlmod function that only contains dependencies that are not available as modules. So their definition is shared between bzlmod and workspace modes.
  • f5d8e9f moves dependencies that are not used by bzlmod after all, from non_module_deps files directly to the WORKSPACE file.
  • Because buildifier is a development dependency of rules_haskell, buildifier targets should not be in the same package as public ones. So they are moved to dedicated buildifier packages.

@ylecornec ylecornec changed the title pre bzlmod refactoring Refactoring for bzlmod Jun 8, 2023
@ylecornec ylecornec marked this pull request as ready for review June 8, 2023 15:38
@ylecornec ylecornec requested a review from avdv as a code owner June 8, 2023 15:38
Base automatically changed from ylecornec/rules_haskell_test to master June 9, 2023 15:11
@dpulls
Copy link

dpulls bot commented Jun 9, 2023

🎉 All dependencies have been resolved !

ylecornec added 4 commits June 9, 2023 18:26
This repository rule only provides rules_haskell dependencies which are not available as bzlmod modules
In bzlmod buildifier is only a dev dependency of rules_haskell. So buildifier targets should not be in the same package as public targets such as `//:distribution`
For consistency with `rules_haskell`, buildifier can also be run with `bazel run //buildifier`
- io_bazel_stardoc is now available as a module
- we will rely on a  module extension from rules_js to install webpack instead of using asterius_dependencies_* rules
@ylecornec ylecornec force-pushed the ylecornec/pre_bzlmod_refactoring branch from f5d8e9f to 8988b52 Compare June 9, 2023 16:26
Copy link
Member

@avdv avdv left a comment

Choose a reason for hiding this comment

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

This is progressing nicely, well done!

@ylecornec ylecornec added the merge-queue merge on green CI label Jun 12, 2023
@ylecornec ylecornec force-pushed the ylecornec/pre_bzlmod_refactoring branch from 494d170 to 3a10e22 Compare June 12, 2023 11:42
@ylecornec ylecornec merged commit ff36492 into master Jun 12, 2023
@ylecornec ylecornec deleted the ylecornec/pre_bzlmod_refactoring branch June 12, 2023 12:30
@mergify mergify bot removed the merge-queue merge on green CI label Jun 12, 2023
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