Skip to content

[Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu)#2987

Merged
ljharb merged 1 commit intoimport-js:mainfrom
joeyguerra:convert-wsl-to-gha
Mar 22, 2024
Merged

[Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu)#2987
ljharb merged 1 commit intoimport-js:mainfrom
joeyguerra:convert-wsl-to-gha

Conversation

@joeyguerra
Copy link
Copy Markdown
Contributor

Converting the appveyor pipeline to GitHub Actions.

Comment thread .eslintrc Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .nycrc Outdated
Comment thread tests/src/core/resolve.js Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .eslintrc Outdated
@joeyguerra
Copy link
Copy Markdown
Contributor Author

I've confirmed that separate runs using Vampire/setup-wsl are disconnected in that you can't use npm in one run when Node is installed in another one. In addition, nvm is unavailable in a run when using ljharb/actions/node/install in a previous run. Thus necessitating the whole script be done in 1 run for the WSL configuration.

I've also taken a different strategy to address the other problems (typically unexpected token errors). Which results in passing tests without having to ignore checks in the utils/ code.

@ljharb would you mind taking another look at this PR?

Copy link
Copy Markdown
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

Thanks, if the tests pass then I'm happy with this approach - I can consolidate things into reusable scripts later :-)

Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml Outdated
Comment thread .github/workflows/native-wsl.yml
@ljharb ljharb changed the title chore: Run tests on Windows in both pwsh and WSL (Ubuntu) [Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu) Mar 22, 2024
@ljharb ljharb merged commit fa60e3d into import-js:main Mar 22, 2024
renovate Bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Sep 4, 2024
| datasource | package              | from   | to     |
| ---------- | -------------------- | ------ | ------ |
| npm        | eslint-plugin-import | 2.29.1 | 2.30.0 |


## [v2.30.0](https://github.com/import-js/eslint-plugin-import/blob/HEAD/CHANGELOG.md#2300---2024-09-02)

##### Added

-   \[`dynamic-import-chunkname`]: add `allowEmpty` option to allow empty leading comments (\[[#2942](import-js/eslint-plugin-import#2942)], thanks \[[@JiangWeixian](https://github.com/JiangWeixian)])
-   \[`dynamic-import-chunkname`]: Allow empty chunk name when webpackMode: 'eager' is set; add suggestions to remove name in eager mode (\[[#3004](import-js/eslint-plugin-import#3004)], thanks \[[@amsardesai](https://github.com/amsardesai)])
-   \[`no-unused-modules`]: Add `ignoreUnusedTypeExports` option (\[[#3011](import-js/eslint-plugin-import#3011)], thanks \[[@silverwind](https://github.com/silverwind)])
-   add support for Flat Config (\[[#3018](import-js/eslint-plugin-import#3018)], thanks \[[@michaelfaith](https://github.com/michaelfaith)])

##### Fixed

-   \[`no-extraneous-dependencies`]: allow wrong path (\[[#3012](import-js/eslint-plugin-import#3012)], thanks \[[@chabb](https://github.com/chabb)])
-   \[`no-cycle`]: use scc algorithm to optimize (\[[#2998](import-js/eslint-plugin-import#2998)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[`no-duplicates`]: Removing duplicates breaks in TypeScript (\[[#3033](import-js/eslint-plugin-import#3033)], thanks \[[@yesl-kim](https://github.com/yesl-kim)])
-   \[`newline-after-import`]: fix considerComments option when require (\[[#2952](import-js/eslint-plugin-import#2952)], thanks \[[@developer-bandi](https://github.com/developer-bandi)])
-   \[`order`]: do not compare first path segment for relative paths (\[[#2682](import-js/eslint-plugin-import#2682)]) (\[[#2885](import-js/eslint-plugin-import#2885)], thanks \[[@mihkeleidast](https://github.com/mihkeleidast)])

##### Changed

-   \[Docs] `no-extraneous-dependencies`: Make glob pattern description more explicit (\[[#2944](import-js/eslint-plugin-import#2944)], thanks \[[@mulztob](https://github.com/mulztob)])
-   \[`no-unused-modules`]: add console message to help debug \[[#2866](import-js/eslint-plugin-import#2866)]
-   \[Refactor] `ExportMap`: make procedures static instead of monkeypatching exportmap (\[[#2982](import-js/eslint-plugin-import#2982)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Refactor] `ExportMap`: separate ExportMap instance from its builder logic (\[[#2985](import-js/eslint-plugin-import#2985)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Docs] `order`: Add a quick note on how unbound imports and --fix (\[[#2640](import-js/eslint-plugin-import#2640)], thanks \[[@MinervaBot](https://github.com/minervabot)])
-   \[Tests] appveyor -> GHA (run tests on Windows in both pwsh and WSL + Ubuntu) (\[[#2987](import-js/eslint-plugin-import#2987)], thanks \[[@joeyguerra](https://github.com/joeyguerra)])
-   \[actions] migrate OSX tests to GHA (\[[ljharb#37](ljharb/eslint-plugin-import#37)], thanks \[[@aks-](https://github.com/aks-)])
-   \[Refactor] `exportMapBuilder`: avoid hoisting (\[[#2989](import-js/eslint-plugin-import#2989)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Refactor] `ExportMap`: extract "builder" logic to separate files (\[[#2991](import-js/eslint-plugin-import#2991)], thanks \[[@soryy708](https://github.com/soryy708)])
-   \[Docs] \[`order`]: update the description of the `pathGroupsExcludedImportTypes` option (\[[#3036](import-js/eslint-plugin-import#3036)], thanks \[[@liby](https://github.com/liby)])
-   \[readme] Clarify how to install the plugin (\[[#2993](import-js/eslint-plugin-import#2993)], thanks \[[@jwbth](https://github.com/jwbth)])
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants