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

Upgrade to Pug 3 #88

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Upgrade to Pug 3 #88

wants to merge 3 commits into from

Conversation

borisovg
Copy link

@borisovg borisovg commented May 3, 2021

  • fix tests by removing broken test
  • upgrade to Pug 3 and upgrade all dependencies
  • configure Travis to test with Node 10, 12 and 14 (Pug 3 supports Node 10+)

closes #86

@borisovg borisovg force-pushed the upgrade-to-pug-3 branch from 7bd3d00 to be4d10c Compare May 3, 2021 18:38
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@0eb8d37). Click here to learn what that means.
The diff coverage is 100.00%.

❗ Current head 7bd3d00 differs from pull request most recent head be4d10c. Consider uploading reports for the commit be4d10c to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             master       #88   +/-   ##
==========================================
  Coverage          ?   100.00%           
==========================================
  Files             ?         1           
  Lines             ?       144           
  Branches          ?         0           
==========================================
  Hits              ?       144           
  Misses            ?         0           
  Partials          ?         0           
Impacted Files Coverage Δ
index.js 100.00% <100.00%> (ø)

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 0eb8d37...be4d10c. Read the comment docs.

@borisovg
Copy link
Author

@ForbesLindesay sorry to bug, is anyone looking at this repo at all?

@Anduh
Copy link

Anduh commented Jun 13, 2021

  • fix tests by removing broken test

@borisovg Did the test report your changes as failing, or is the test actually broken? Did it break before you updated the dependencies & made the changes, or afterwards? Removing a test bc it's broken doesn't sound like a fix per se.

Updating the pug-cli version is probably also needed for this change to automatically propagate, like #87 did. I've made PR to your update that updates the version & the changelog.

@Anduh
Copy link

Anduh commented Jun 13, 2021

@TimothyGu can you take a look at this? pub-cli haven't been updated to PUG 3 yet, and this PR is hopefully all that's needed.

@borisovg
Copy link
Author

@Anduh the test was already failing before I made any changes and I didn’t see anything in the code specific to what that test was supposedly testing (rejecting UTF new lines)

@Anduh Anduh mentioned this pull request Jun 13, 2021
@Anduh
Copy link

Anduh commented Jun 21, 2021

@borisovg I created the @anduh/pug-cli-package based on your update, so that there now exists an alternative package to this stagnant one.

Are you interested in becoming a co-maintainer of it? I haven't set up the Codecov or Travis CI stuff yet, but will look into it, if the original pug-cli doesn't get any attention by the authors.

I changed it's command to pug3, so one can have both pug-cli (using pug) & @anduh/pug-cli (using pug3) installed at the same time, without conflicting commands.

@mvorisek
Copy link

mvorisek commented Oct 5, 2022

Is there any feedback left to address or can this PR be merged?

kaosf added a commit to kaosf/kaosfield that referenced this pull request Feb 10, 2023
To fix pugjs/pug#3312

Old original `pug-cli` depends on `[email protected]`. `@anduh/pug-cli`
is the fork to fix this problem.

ref. pugjs/pug-cli#88 (comment)
svallory added a commit to tokilabs/pug3-cli that referenced this pull request Apr 23, 2023
…#4 from tokilabs/merging-forks)

Here's a list of the merged contributions:

- @Anduh at [anduh/pug-cli](https://github.com/anduh/pug-cli)
  - Update Pug to version 3.0.2 (Closes pugjs#88)
- @anthony-tron at [anthony-tron/pug-cli](https://github.com/anthony-tron/pug-cli)
  - Hot reload of options file and YAML in `-O | --obj` option (Closes pugjs#89)
- @lazdmx at https://github.com/lazdmx/pug-cli
  - Allow symlink as input 2d6c334 (Closes pugjs#80) [2d6c334](lazdmx@2d6c334)
- @funai at [funai/pug-cli](https://github.com/funai/pug-cli)
  - [Allow templates to use YAML front-matter](4d5b6e6)
  - [Inject `extends` from  frontmatter `layout`](4d5b6e6#diff-e727e4bdf3657fd1d798edcd6b099d6e092f8573cba266154583a746bba0f346R260)
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.

update pug version
4 participants