Skip to content

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

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

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Aug 20, 2024

Reverts #4350

@dbkr dbkr added the T-Task Tasks for the team like planning label Aug 20, 2024
@dbkr dbkr marked this pull request as ready for review August 20, 2024 10:41
@dbkr dbkr requested a review from a team as a code owner August 20, 2024 10:41
@dbkr dbkr requested review from t3chguy and robintown August 20, 2024 10:41
@MidhunSureshR MidhunSureshR self-requested a review August 20, 2024 10:49
@dbkr dbkr added this pull request to the merge queue Aug 20, 2024
@richvdh richvdh removed this pull request from the merge queue due to a manual request Aug 20, 2024
@richvdh richvdh added this pull request to the merge queue Aug 20, 2024
@richvdh
Copy link
Member

richvdh commented Aug 20, 2024

This is a temporary revert while we cut this release. we'll unrevert it for this afternoon's RC

Merged via the queue into develop with commit 8214fd7 Aug 20, 2024
31 checks passed
@richvdh richvdh deleted the revert-4350-prerelease-fix branch August 20, 2024 11:25
richvdh added a commit that referenced this pull request Aug 20, 2024
github-merge-queue bot pushed a commit that referenced this pull request Aug 20, 2024
* Reapply "Add "type" = "module" to ensure it is present (#4350)" (#4352)

This reverts commit 8214fd7.

* Mark prettier config file as CommonJS

I *think* this will fix a problem with the release process in which we saw an
error:

```
Error:  Invalid configuration for file "/home/runner/work/matrix-js-sdk/matrix-js-sdk/package.json":
Error:  module is not defined in ES module scope
Error:  This file is being treated as an ES module because it has a '.js' file extension and '/home/runner/work/matrix-js-sdk/matrix-js-sdk/package.json' contains "type": "module". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants