Skip to content

Skip HLS on trivial changes#811

Merged
palas merged 1 commit intomasterfrom
skip-hls-on-trivial-changes
Apr 10, 2025
Merged

Skip HLS on trivial changes#811
palas merged 1 commit intomasterfrom
skip-hls-on-trivial-changes

Conversation

@palas
Copy link
Copy Markdown
Contributor

@palas palas commented Apr 10, 2025

Changelog

- description: |
    Skip HLS checks on trivial changes
  type:
  - maintenance    # not directly related to the code

Context

Doing releases doesn't affect whether HLS works or not. For that reason, we want to skip the check when it doesn't make sense. This PR introduces an exception for when only markdown is changed and for when only the version in a cabal file is changed.

How to trust this PR

It is the same as in the CLI PR: IntersectMBO/cardano-cli#1132

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. See Running tests for more details
  • Self-reviewed the diff

@palas palas self-assigned this Apr 10, 2025
@palas palas requested review from a team, Jimbo4350, carbolymer and newhoggy as code owners April 10, 2025 12:19
@palas palas enabled auto-merge April 10, 2025 12:27
@palas palas added this pull request to the merge queue Apr 10, 2025
Merged via the queue into master with commit 457b7eb Apr 10, 2025
29 checks passed
@palas palas deleted the skip-hls-on-trivial-changes branch April 10, 2025 13:56
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.

3 participants