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

Initial branch #1

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Initial branch #1

wants to merge 15 commits into from

Conversation

ehoogerbeets
Copy link
Contributor

@ehoogerbeets ehoogerbeets commented Oct 28, 2024

  • added circleci config that runs only the tests for the part of the monorepo that you have changed (may not work fully yet)
  • added a README.md that describes how to use this circleci stuff
  • builds can be seen at https://app.circleci.com/pipelines/github/iLib-js/ilib-mono
  • added the first cut of ilib-lint. This has not been modified from v2.5.0 yet and does not have branches and such copied yet. It is just there to test the circleci stuff. I'll add the branches and such later.
  • ilib-lnit is in the packages directory. If we choose a package manager that needs it in another directory, we can move it then

- run:
name: Setup
command: |
rm -rf node_modules package-lock.json

Choose a reason for hiding this comment

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

Suggested change
rm -rf node_modules package-lock.json
rm -rf node_modules package-lock.json yarn.lock

name: Setup
command: |
rm -rf node_modules package-lock.json
npm install

Choose a reason for hiding this comment

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

Is there a reason for using npm instead of yarn?
Shall we use yarn instead?
Since we're creating brand new repo, I'd love to stick to modern stack whenever possible.

Copy link

@nmkedziora nmkedziora Oct 29, 2024

Choose a reason for hiding this comment

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

Few quotes from yarn creators:

The npm client installs dependencies into the node_modules directory non-deterministically. This means that based on the order dependencies are installed, the structure of a node_modules directory could be different from one person to another. These differences can cause “works on my machine” bugs that take a long time to hunt down.

Yarn resolves these issues around versioning and non-determinism by using lockfiles and an install algorithm that is deterministic and reliable.

(...) Yarn is able to parallelize operations, which maximizes resource utilization and makes the install process faster.

Source: Yarn: A new package manager for JavaScript

@@ -0,0 +1,184 @@
# copied from https://github.com/circle-makotom/circle-advanced-setup-workflow/blob/main/.circleci/config.yml
Copy link

@nmkedziora nmkedziora Oct 29, 2024

Choose a reason for hiding this comment

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

Do we need all of these? How about starting with minimal setup and adding things as we go, once they're required?

workflows:
version: 2
test-all-node-version:
jobs:
Copy link

@nmkedziora nmkedziora Oct 29, 2024

Choose a reason for hiding this comment

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

Why those Node.js versions and not the others?

Choose a reason for hiding this comment

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

Why browsers variants?

docker:
- image: cimg/base:stable
steps:
- run: echo "No updates have been made to any of the subpackages"

Choose a reason for hiding this comment

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

Suggested change
- run: echo "No updates have been made to any of the subpackages"
- run: echo "No updates have been made to any of the packages"

@@ -0,0 +1,8 @@
version: 2.1

Choose a reason for hiding this comment

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

When is this used? I cannot find usages with search.

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.

2 participants