Skip to content
This repository was archived by the owner on Jun 15, 2023. It is now read-only.

Fix Windows build and beautify + format ci.yml #580

Merged
merged 6 commits into from
Jun 24, 2022
Merged

Fix Windows build and beautify + format ci.yml #580

merged 6 commits into from
Jun 24, 2022

Conversation

cknitt
Copy link
Member

@cknitt cknitt commented Jun 24, 2022 •
edited
Loading

  • Do not override default OPAM repo for Windows anymore. This fixes the Windows build.
  • Beautify and format ci.yml to bring it in line with the rescript-compiler repo.

@cknitt cknitt changed the title ocaml/setup-ocaml@v2.0.3 -> ocaml/setup-ocaml@v2 Fix Windows build and beautify + format ci.yml Jun 24, 2022
@cknitt
Copy link
Member Author

cknitt commented Jun 24, 2022

Ah, there seems to be remaining issue on Windows with ocamlformat.

@cknitt cknitt merged commit fed1a27 into rescript-lang:master Jun 24, 2022
@cknitt cknitt deleted the setup-ocaml-v2 branch June 24, 2022 19:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cristianoc cristianoc cristianoc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cknitt @cristianoc