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

Ensure "type" = "module" ES declaration in pre-release.sh #4349

Closed
wants to merge 1 commit into from
Closed

Ensure "type" = "module" ES declaration in pre-release.sh #4349

wants to merge 1 commit into from

Conversation

BLCK-B
Copy link
Contributor

@BLCK-B BLCK-B commented Aug 15, 2024

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • Sign-off given on the changes (see CONTRIBUTING.md).

Attempts to fix the issue #4347. Added a line in scripts/pre-release.sh that should add "type" = "module" to package.json. Analogous to 432aaa4.

Reasoning:
CI order goes Release Process > Release Make > pre-release.sh. It does not involve the other release script with the previous fix.

Unfortunately, I can't test or verify. The CI workflows are too complex.

Signed-off-by: Jonas Black [email protected]

@BLCK-B BLCK-B requested a review from a team as a code owner August 15, 2024 14:20
@BLCK-B BLCK-B requested review from dbkr and t3chguy August 15, 2024 14:20
@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Aug 15, 2024
Copy link
Contributor

Thanks for opening this pull request, unfortunately we do not accept contributions from the main branch of your fork, please re-open once you switch to an alternative branch for everyone's sanity. See https://github.com/matrix-org/matrix-js-sdk/blob/develop/CONTRIBUTING.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant