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

fix: eslint rules around imports #92

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

SimeonC
Copy link
Contributor

@SimeonC SimeonC commented Oct 25, 2023

  • Fixed more edge cases for shortestImport.
  • Added naming convention for imports
  • made lint executor set type aware options manually

This took me over a day to figure out everything and my eyes are glazing over 😅 .
if you see somewhere I missed a refactor opportunity please let me know 🙇

📦 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]

@SimeonC SimeonC added the patch Increment the patch version when merged label Oct 25, 2023
@SimeonC SimeonC force-pushed the fix-shortest-import-and-eslint-import-naming branch from 55207ef to c106c1c Compare October 25, 2023 02:25
@github-actions
Copy link

github-actions bot commented Oct 25, 2023

Unit Test Results

    1 files      3 suites   22s ⏱️
185 tests 185 ✔️ 0 💤 0
195 runs  195 ✔️ 0 💤 0

Results for commit d8ffe0b.

♻️ This comment has been updated with latest results.

@SimeonC SimeonC force-pushed the fix-shortest-import-and-eslint-import-naming branch 3 times, most recently from 2fed5cc to 07a1450 Compare October 25, 2023 02:37
@SimeonC SimeonC enabled auto-merge (rebase) October 25, 2023 02:41
@SimeonC SimeonC force-pushed the fix-shortest-import-and-eslint-import-naming branch 3 times, most recently from 2f3e32a to 0aad08f Compare October 25, 2023 06:03
SashaShostyr
SashaShostyr previously approved these changes Oct 25, 2023
@SimeonC SimeonC force-pushed the fix-shortest-import-and-eslint-import-naming branch from 0aad08f to aff75ad Compare October 26, 2023 05:44
* Fixed more edge cases for shortestImport.
* Added naming convention for imports
* made lint executor set type aware options manually
@SimeonC SimeonC force-pushed the fix-shortest-import-and-eslint-import-naming branch from aff75ad to d8ffe0b Compare October 26, 2023 05:54
@SimeonC SimeonC merged commit 22b4e2a into main Oct 26, 2023
5 checks passed
@SimeonC SimeonC deleted the fix-shortest-import-and-eslint-import-naming branch October 26, 2023 08:58
@tablecheck-public-automation

@tablecheck-public-automation tablecheck-public-automation added the released This issue/pull request has been released. label Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch 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