Skip to content

ci: fix invalid workflow - #1685

Merged
Aleksander-Karlsson merged 1 commit into
mainfrom
ci/fix-publish-docs
Jul 22, 2026
Merged

ci: fix invalid workflow#1685
Aleksander-Karlsson merged 1 commit into
mainfrom
ci/fix-publish-docs

Conversation

@Aleksander-Karlsson

Copy link
Copy Markdown
Contributor

Type of Work

  • Patch: X.Y.Z+1. NEGLIGIBLE visible changes, does not change input or output - OR changes behaviour. Use chore:, refactor: etc
  • Minor: X.Y+1.Z. Minor changes, might ADD new input (YAML), or other backwards-compatible changes. Use feat:, fix:
  • Major: X+1.Y.Z. Major and most likely BREAKING changes, wo. backwards compatibility, or removing temporary backwards compatibility functionality. Use ! or BREAKING:.

See here (internal): https://github.com/equinor/ecalc-internal/discussions/1044

Have you remembered and considered?

  • IF FEAT: I have remembered to update documentation
  • IF FIX OR FEAT: I have remembered to update manual changelog (docs/drafts/next.draft.md)
  • IF BREAKING: I have remembered to update migration guide (docs/docs/migration_guides/)
  • IF BREAKING: I have committed with BREAKING: in footer or ! in header
  • I have added tests (if not, comment why)
  • I have used conventional commits syntax (if you squash, make sure that conventional commit is used)
  • I have included the Github issue nr in the footer!

What is this PR all about?

What else did you consider?

Between the lines?

@Aleksander-Karlsson
Aleksander-Karlsson marked this pull request as ready for review July 22, 2026 09:56
@Aleksander-Karlsson
Aleksander-Karlsson requested a review from a team as a code owner July 22, 2026 09:56
chore: use secret.GITHUB_TOKEN instead
@Aleksander-Karlsson
Aleksander-Karlsson merged commit 297810e into main Jul 22, 2026
25 checks passed
@Aleksander-Karlsson
Aleksander-Karlsson deleted the ci/fix-publish-docs branch July 22, 2026 11:11
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