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

Integrations setup #4354

Merged
merged 17 commits into from
May 18, 2021
Merged

Integrations setup #4354

merged 17 commits into from
May 18, 2021

Conversation

RobinMalfait
Copy link
Contributor

@RobinMalfait RobinMalfait commented May 14, 2021

Integrations tested:

  • webpack 4
  • webpack 5
  • rollup.js
  • vite
  • PostCSS CLI
  • Parcel (Broken tests - but expected)

@codecov-commenter
Copy link

codecov-commenter commented May 14, 2021

Codecov Report

Merging #4354 (5181e50) into master (6be7976) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4354      +/-   ##
==========================================
- Coverage   71.85%   71.84%   -0.02%     
==========================================
  Files         220      220              
  Lines        4769     4806      +37     
  Branches      790      805      +15     
==========================================
+ Hits         3427     3453      +26     
- Misses       1262     1267       +5     
- Partials       80       86       +6     
Impacted Files Coverage Δ
src/jit/lib/rewriteTailwindImports.js
src/jit/lib/normalizeTailwindDirectives.js 71.42% <0.00%> (ø)
src/jit/lib/setupContext.js 77.88% <0.00%> (+0.05%) ⬆️
src/util/pluginUtils.js 60.91% <0.00%> (+0.30%) ⬆️
src/jit/lib/expandTailwindAtRules.js 83.58% <0.00%> (+2.23%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6be7976...5181e50. Read the comment docs.

@RobinMalfait RobinMalfait force-pushed the integrations-setup branch 7 times, most recently from 0c2fffd to cb1cc69 Compare May 17, 2021 16:45
@RobinMalfait RobinMalfait force-pushed the integrations-setup branch 2 times, most recently from 8efde3f to b6638d5 Compare May 18, 2021 14:46
We will wait for the correct stdout/stderr output, once we know that we
can request the fresh css, we will fetch it and make assertions
accordingly.

Port is currently hardcoded, maybe we should use a packaage to ensure
that we use a free port.
@adamwathan adamwathan deleted the integrations-setup branch May 18, 2021 15: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.

None yet

3 participants