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: Use @eik/common to load config in a project #84

Merged
merged 5 commits into from
Mar 2, 2021
Merged

Conversation

trygve-lie
Copy link
Contributor

@trygve-lie trygve-lie commented Feb 19, 2021

This makes use of the logic found in @eik/common to config in a project. This should now align with how the client and other plugins do this.

The API surface is also changed. Instead of taking an Import Map as the configuration option it can now take an options object with the following arguments:

  • path for defining an path to a eik.json other than in the root of the project
  • urls for defining URLs to custom Import Maps. If a Import Map rule exist in both the config and in a Import Map defined through urls, the rule defined in urls take precedence.
  • maps for defining custom Import Maps as objects. If a Import Map rule exist in both the urls and in a Import Map defined through maps, the rule defined in maps take precedence.

This align the config to be similar to the other build tool plugins.

Trygve Lie and others added 3 commits February 19, 2021 17:10
README.md Outdated Show resolved Hide resolved
src/plugin.js Outdated Show resolved Hide resolved
src/plugin.js Outdated Show resolved Hide resolved
test/plugin.js Outdated Show resolved Hide resolved
"semantic-release": "17.3.9",
"tap": "14.11.0"
},
"dependencies": {
"@eik/common": "4.0.0-next.4",

Choose a reason for hiding this comment

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

Wondering if using this version is going to cause any issues? Maybe safer to go with v2 until v4 is released?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah. I am not 100% sure how to fix this one. Semantic Release did just bump two major version without any good reason. Even reported that as a bug: semantic-release/semantic-release#1809

Any good ideas on how to solve it?

Choose a reason for hiding this comment

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

Perhaps roll the version being used here back to v2 for now?

If you want to fix release of v4 instead of v3 could you manually change the version to v3.0.0-next.1 and then update the dist tag? When releasing v4.0.0-next one would need to manually jump over to 4.0.0-next.5 I assume. Might not be worth it to fix it.

@trygve-lie trygve-lie merged commit 371dcda into next Mar 2, 2021
@trygve-lie trygve-lie deleted the use-common branch March 2, 2021 13:09
github-actions bot pushed a commit that referenced this pull request Mar 2, 2021
# [3.0.0-next.2](v3.0.0-next.1...v3.0.0-next.2) (2021-03-02)

### Features

* Use @eik/common to load config in a project ([#84](#84)) ([371dcda](371dcda))

### BREAKING CHANGES

* Use logic in @eik/common to load config from eik.json or package.json

* chore: Adjust for PR comments

* chore: Remove extra whitespace

Co-authored-by: Trygve Lie <trygve.lie@finn.no>
@github-actions
Copy link

github-actions bot commented Mar 2, 2021

🎉 This PR is included in version 3.0.0-next.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Aug 13, 2021
# [3.0.0](v2.0.4...v3.0.0) (2021-08-13)

### Features

* Rename module to @eik/postcss-plugin ([#83](#83)) ([7efa758](7efa758))
* Use @eik/common to load config in a project ([#84](#84)) ([371dcda](371dcda))

### BREAKING CHANGES

* Use logic in @eik/common to load config from eik.json or package.json

* chore: Adjust for PR comments

* chore: Remove extra whitespace

Co-authored-by: Trygve Lie <trygve.lie@finn.no>
* Rename module to @eik/postcss-plugin

Co-authored-by: Trygve Lie <trygve.lie@finn.no>
@github-actions
Copy link

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants