Skip to content

Workflow updates #1903

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

Merged
merged 12 commits into from
Mar 6, 2024
Merged

Workflow updates #1903

merged 12 commits into from
Mar 6, 2024

Conversation

rajsite
Copy link
Member

@rajsite rajsite commented Mar 5, 2024

Pull Request

🤨 Rationale

  • Updated fetch-depth: 0 comment from being needed by beachball to being needed by chromatic.
  • Updated renovate to make a single PR for npm updates that we can let update to latest.
    • Angular and it's related packages are ignored
      • ^@angular, ng-packagr, zone.js, typescript
    • Pinned packages are ignored
      • @microsoft/fast-foundation and @microsoft/fast-react-wrapper due to fast issue
      • comlink as it is partially vendored into workers and needs a controlled and pinned update
      • remark-gfm as updating breaks mdx
  • Removed npm version specific install that was now downgrading the npm version accidentally.
  • Removed file permission workaround no longer needed to prevent warnings.
  • Updated the beachball patch-package

👩‍💻 Implementation

See above.

🧪 Testing

Relying on CI.

✅ Checklist

  • I have updated the project documentation to reflect my changes or determined no changes are needed.

@rajsite rajsite marked this pull request as ready for review March 6, 2024 02:13
@rajsite rajsite requested a review from jattasNI as a code owner March 6, 2024 02:13
@rajsite rajsite enabled auto-merge (squash) March 6, 2024 18:09
@rajsite rajsite merged commit 0098b27 into main Mar 6, 2024
@rajsite rajsite deleted the workflow-updates branch March 6, 2024 18:27
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