Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit c7e84db

Browse files
committedDec 29, 2021
CI: flags: post-review fxs
1 parent dc5c389 commit c7e84db

File tree

1 file changed

+5
-14
lines changed

1 file changed

+5
-14
lines changed
 

‎.github/workflows/flags.yml

+5-14
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,6 @@ jobs:
3636
, ".circleci/**"
3737
, "**/stack*.yaml"
3838
]'
39-
# If we only change ghcide downstream packages we have not test ghcide itself
40-
- id: skip_ghcide_check
41-
uses: fkirc/skip-duplicate-actions@v3.4.1
42-
with:
43-
cancel_others: false
44-
paths_ignore: '[ "hls-test-utils/**"
45-
, "plugins/**"
46-
, "src/**"
47-
, "exe/**"
48-
, "test/**"
49-
, "shake-bench/**"
50-
]'
5139

5240
flags:
5341
if: needs.pre_job.outputs.should_skip != 'true'
@@ -131,6 +119,10 @@ jobs:
131119
${{ env.cache-name }}-${{ runner.os }}-${{ matrix.ghc }}-
132120
${{ env.cache-name }}-${{ runner.os }}-
133121
122+
# To ensure we get the lastest hackage index and not relying on haskell action logic
123+
- if: steps.compiled-deps.outputs.cache-hit != 'true'
124+
run: cabal update
125+
134126
- name: Build `hls-graph` with flags
135127
run: cabal v2-build hls-graph --flags="pedantic embed-files stm-stats"
136128

@@ -143,9 +135,8 @@ jobs:
143135
- name: Build `hls-test-utils` with flags
144136
run: cabal v2-build hls-test-utils --flags="pedantic"
145137

146-
# repeating builds to workaround segfaults in windows and ghc-8.8.4
147138
- name: Build
148-
run: cabal v2-build ghcide --flags="ghc-patched-unboxed-bytecode test-exe executable bench-exe" || cabal v2-build ghcide --flags="ghc-patched-unboxed-bytecode test-exe executable bench-exe" || cabal v2-build ghcide --flags="ghc-patched-unboxed-bytecode test-exe executable bench-exe"
139+
run: cabal v2-build ghcide --flags="ghc-patched-unboxed-bytecode test-exe executable bench-exe"
149140

150141
flags_post_job:
151142
if: always()

0 commit comments

Comments
 (0)
Please sign in to comment.