Skip to content

Conversation

@TomStrepsil
Copy link
Contributor

@TomStrepsil TomStrepsil commented Feb 11, 2025

Issue

resolves #24

Details

Update to fixed version of danger-js, moving the common-js code back to ESM, via removing the transpilation hack introduced when the project was open-sourced.

See successful danger check run.

Scout Rule

  • lint some punctuation in the main CHANGELOG.md

CheckList

  • PR starts with [ISSUE_ID].
  • Has been tested (where required) before merge to main.

TomStrepsil and others added 6 commits December 24, 2024 17:31
* rename to proper module namespace

* update docs links

* update versions

* web toggle point in readme title

* fixup changelog from revised 0.x range

* 2.0.0 -> 0.5.0 in oss version scheme

* fix broken link syntax in CHANGELOG

* consistent quoting

* more version history issues

* fixup module name in jsdoc

* add web
remove sdkInstanceProvider

* remove SDKInstanceProvider

* fixup jsdoc dedupe

* tweak

* clarity re: ssr package

* casing etc
@TomStrepsil TomStrepsil changed the title Move to new version of danger-js [24] Move to new version of danger-js Feb 11, 2025
@TomStrepsil TomStrepsil changed the title [24] Move to new version of danger-js [24] Move to new version of danger-js, allowing a move back to ESM for first-party rules Mar 2, 2025
TomStrepsil and others added 4 commits March 4, 2025 10:33
* update workflows

* version

* typo

* update chromium linux snaps

* versions for serve update

* package.json repository field

* update root package.lock

* bugs & directories/doc fields

* fix changelog

---------

Co-authored-by: Tom Pereira <[email protected]>
TomStrepsil pushed a commit to TomStrepsil/web-toggle-point that referenced this pull request Mar 6, 2025
TomStrepsil added a commit to TomStrepsil/web-toggle-point that referenced this pull request Apr 10, 2025
@TomStrepsil TomStrepsil force-pushed the fix/revert-danger-to-esm branch from 95a6d9f to 3d85137 Compare April 10, 2025 19:53
Copy link

@asos-dominicjomaa asos-dominicjomaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mmmm esm

@TomStrepsil TomStrepsil requested a review from a team as a code owner July 14, 2025 14:56
@asos-dominicjomaa asos-dominicjomaa merged commit 5593895 into ASOS:main Jul 14, 2025
10 checks passed
@TomStrepsil TomStrepsil deleted the fix/revert-danger-to-esm branch October 21, 2025 09:26
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.

Allow ESM in danger-js code

3 participants