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

feat: add a wrapper around @nx/js generator and fix some eslint issues #93

Merged
merged 5 commits into from
Nov 2, 2023

Conversation

SimeonC
Copy link
Contributor

@SimeonC SimeonC commented Oct 31, 2023

Wrapping some other libraries so we can easily create new vite libraries in one command.

Also updating Eslint config formats and quality to help with that.

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @tablecheck/[email protected]
npm install @tablecheck/[email protected]
npm install @tablecheck/[email protected]
# or 
yarn add @tablecheck/[email protected]
yarn add @tablecheck/[email protected]
yarn add @tablecheck/[email protected]

@github-actions
Copy link

github-actions bot commented Oct 31, 2023

Unit Test Results

    1 files      3 suites   25s ⏱️
193 tests 193 ✔️ 0 💤 0
211 runs  211 ✔️ 0 💤 0

Results for commit ac4fb82.

♻️ This comment has been updated with latest results.

@SimeonC SimeonC added the minor Increment the minor version when merged label Oct 31, 2023
@SimeonC SimeonC force-pushed the new-nx-plugin branch 5 times, most recently from 4ce04a3 to 0f99500 Compare November 1, 2023 03:10
Wrapping some other libraries so we can easily create new vite libraries in one command.
Allows us to use the eslint configs more flexibly. Upgrades the quality generator to allow selecting which preset - required for NX cypress project usage.
Resolves the following scenario;
* `../Common/ShopSelect`
* `~/Common/ShopSelect`

These are considered “same length” but we want to prefer the `~` alias path as it’s easier to read than the relative path.
Found and fixed another preferred alias’ bug.
@SimeonC SimeonC changed the title feat: add a wrapper around @nx/js generator feat: add a wrapper around @nx/js generator and fix some eslint issues Nov 2, 2023
@SimeonC SimeonC force-pushed the new-nx-plugin branch 3 times, most recently from eec7341 to b8e6e74 Compare November 2, 2023 04:11
There is a technically “breaking change” in here, but I know no-one is using the undocumented preferredAlias options yet.
@SimeonC SimeonC merged commit a9953fe into main Nov 2, 2023
5 checks passed
@SimeonC SimeonC deleted the new-nx-plugin branch November 2, 2023 07:54
@tablecheck-public-automation

1 similar comment
@tablecheck-public-automation

@tablecheck-public-automation tablecheck-public-automation added the released This issue/pull request has been released. label Nov 2, 2023
@tablecheck-public-automation

2 similar comments
@tablecheck-public-automation

@tablecheck-public-automation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants