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

chore(deps): bump postcss from 8.4.21 to 8.4.31 in /docs #3685

Merged
merged 9 commits into from
Oct 31, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Oct 7, 2023

Bumps postcss from 8.4.21 to 8.4.31.

Release notes

Sourced from postcss's releases.

8.4.31

8.4.30

8.4.29

8.4.28

  • Fixed Root.source.end for better source map (by @​romainmenke).
  • Fixed Result.root types when process() has no parser.

8.4.27

  • Fixed Container clone methods types.

8.4.26

  • Fixed clone methods types.

8.4.25

8.4.24

  • Fixed Plugin types.

8.4.23

  • Fixed warnings in TypeDoc.

8.4.22

Changelog

Sourced from postcss's changelog.

8.4.31

8.4.30

  • Improved source map performance (by Romain Menke).

8.4.29

  • Fixed Node#source.offset (by Ido Rosenthal).
  • Fixed docs (by Christian Oliff).

8.4.28

  • Fixed Root.source.end for better source map (by Romain Menke).
  • Fixed Result.root types when process() has no parser.

8.4.27

  • Fixed Container clone methods types.

8.4.26

  • Fixed clone methods types.

8.4.25

8.4.24

  • Fixed Plugin types.

8.4.23

  • Fixed warnings in TypeDoc.

8.4.22

  • Fixed TypeScript support with node16 (by Remco Haszing).
Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Oct 7, 2023
@dependabot dependabot bot requested a review from julienrbrt as a code owner October 7, 2023 15:15
@dependabot dependabot bot added component:js Pull requests that update Javascript code skip-changelog Don't check changelog for new entries labels Oct 7, 2023
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs 📖 docs labels Oct 7, 2023
@julienrbrt
Copy link
Member

Weird that CI tries to deploy on PRs.
I think that step should by skipped on PRs?

@julienrbrt julienrbrt self-assigned this Oct 8, 2023
@jeronimoalbi
Copy link
Member

Weird that CI tries to deploy on PRs. I think that step should by skipped on PRs?

It's necessary to check that the dependency update doesn't break the docs I think.
In any case it seems there is a configuration issue here with Firebase.

@mazzy89

@mazzy89
Copy link
Contributor

mazzy89 commented Oct 9, 2023

@jeronimoalbi I will check. Please bear with me. It was not me configuring the CI job deploying to Firebase so need to dive deep there first.

@julienrbrt
Copy link
Member

Weird that CI tries to deploy on PRs. I think that step should by skipped on PRs?

It's necessary to check that the dependency update doesn't break the docs I think.

In any case it seems there is a configuration issue here with Firebase.

@mazzy89

Building the docs in CI definitely makes sense, and there is a step for it. What does not make sense for me is the deploy step.

@julienrbrt julienrbrt assigned mazzy89 and unassigned julienrbrt Oct 9, 2023
@mazzy89
Copy link
Contributor

mazzy89 commented Oct 9, 2023

What does not make sense for me is the deploy step.

It builds a preview in a staging Firebase of how it would look like the docs when changes are committed. I think it makes totally sense. We do this extensively in also other repositories and it helps UI engineers to see live changes rather locally. Maybe we should not do it for version bumps but only for actual changes to the docs contents, structure, etc...

@julienrbrt
Copy link
Member

julienrbrt commented Oct 9, 2023

What does not make sense for me is the deploy step.

It builds a preview in a staging Firebase of how it would look like the docs when changes are committed. I think it makes totally sense. We do this extensively in also other repositories and it helps UI engineers to see live changes rather locally. Maybe we should not do it for version bumps but only for actual changes to the docs contents, structure, etc...

Ahh I see. Then indeed it makes sense! I thought it would directly deploy the docs to prod 😅. Thanks for the clarification.

@mazzy89
Copy link
Contributor

mazzy89 commented Oct 9, 2023

Speaking about the issue of the Deploy step, it seems that the secrets is not filled
https://github.com/ignite/cli/blob/88ffe6208f81e158b096f8aeb78b8778075c875b/.github/workflows/docs-deploy-preview.yml#L42C1-L43C1
The job complains with the error:

Error: Input required and not supplied: firebaseServiceAccount

Unfortunately I do not have privileges to check repositories secrets. So this require escalation to someone who can access them.

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.21 to 8.4.31.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.4.21...8.4.31)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/docs/postcss-8.4.31 branch from 2c55aad to 9d953e5 Compare October 10, 2023 07:21
@github-actions
Copy link
Contributor

github-actions bot commented Oct 12, 2023

Visit the preview URL for this PR (updated for commit b763234):

https://igntservices-docs--pr3685-dependabot-npm-and-y-7xmbl86n.web.app

(expires Tue, 07 Nov 2023 09:51:14 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 95379efd94dd497aaa37c2d0354e6e2cafca5ec5

@julienrbrt julienrbrt merged commit 54bb6b4 into main Oct 31, 2023
26 checks passed
@julienrbrt julienrbrt deleted the dependabot/npm_and_yarn/docs/postcss-8.4.31 branch October 31, 2023 23:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ci CI/CD workflow and automated jobs. component:configs component:js Pull requests that update Javascript code dependencies Pull requests that update a dependency file skip-changelog Don't check changelog for new entries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants