Skip to content

Conversation

jzolnowski
Copy link

Use npm as mentioned in option 2.

  • migrate to npm
  • update README
  • remove Scripts folder
  • clean invalid references

@jzolnowski
Copy link
Author

@gkalpak PR ready for final review

@jzolnowski
Copy link
Author

@gkalpak ready for final review

@Splaktar
Copy link

@jzolnowski please rebase and resolve conflicts.

@jzolnowski
Copy link
Author

Branch up-to-date with main

@gkalpak
Copy link

gkalpak commented Apr 1, 2023

@jzolnowski, please make sure all commits are signed.

@jzolnowski jzolnowski force-pushed the feature/migration-to-npm branch from 7e363e2 to 38c1e5a Compare April 3, 2023 21:22
jzolnowski and others added 6 commits April 3, 2023 23:28
- add tests for index, test-component, and test-directive
- add prettier as dev-dependency, reorder imports, restructure gitignore
- add a separate section for idea and playwright files and folders
- fix ASP.NET error when running the app
- add npm-check to the devDeps
- fix lint issues
@jzolnowski jzolnowski force-pushed the feature/migration-to-npm branch from 38c1e5a to 63c6123 Compare April 3, 2023 21:31
@jzolnowski
Copy link
Author

@jzolnowski, please make sure all commits are signed.

All commits signed

@jzolnowski
Copy link
Author

Branch up-to-date with main

Jaroslaw Zolnowski added 2 commits April 5, 2023 14:36
…e-lock-json, set nvmrc node to v16, remove Playwright e2e tests related fodlers from .gitignore
@jzolnowski
Copy link
Author

engines.node field in package.json set to v16

@gkalpak gkalpak merged commit 4cf220a into main Apr 7, 2023
@gkalpak gkalpak deleted the feature/migration-to-npm branch April 7, 2023 17:21
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.

3 participants