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

[OPS-1448] Replace buildkite with github actions #134

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

Sereja313
Copy link
Member

Problem: The current buildkite-based ci is being replaced with github actions.

Solution: Add github ci config, remove buildkite config.

@Sereja313 Sereja313 requested a review from a team September 27, 2023 08:11
@piegamesde
Copy link
Member

Closes #119

@rvem
Copy link
Contributor

rvem commented Sep 27, 2023

Don't forget to remove buildkite check from required and add check:)

run: nix shell .#packages.x86_64-linux.reuse -c reuse lint

- name: build
run: nix build .#nixfmt-static
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably should check that .#nixfmt-webdemo and .#nixfmt-deriver build as well though I'm not sure what the latter is. Maybe .#awscli too to be sure that the deploy action will work after nixpkgs upgrades. Put them all in separate checks if possible.

Copy link
Member Author

Choose a reason for hiding this comment

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

Added checks for nixfmt-webdemo and awscli. nixfmt-deriver is a nix expression generated by cabal2nix, it was pushed into cachix, which I guess no one uses. I also added hlint and stylish-haskell checks, but decided not to apply the changes to avoid large rebases in other pull requests.

Problem: The current buildkite-based ci is being replaced with github actions.

Solution: Add github ci config, remove buildkite config.
@Sereja313 Sereja313 merged commit 6cdba9e into master Oct 9, 2023
2 checks passed
@Sereja313 Sereja313 deleted the sereja/OPS-1448-bk-to-ga branch October 9, 2023 17:18
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.

4 participants