Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions .chronus/changes/upgrade-rush-pnpm-2026-6-9-17-22.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
---
changeKind: fix
packages:
- "@azure-tools/extension"
---

Remove unused npm dependency; simplify to always use Yarn
8 changes: 8 additions & 0 deletions .chronus/changes/upgrade-rush-pnpm-2026-6-9-18-6.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
---
changeKind: fix
packages:
- "@autorest/core"
- "autorest"
---

Update ExtensionManager.Create call after removing packageManagerType param
36 changes: 36 additions & 0 deletions .chronus/config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
baseBranch: main
baseRemote: https://github.com/Azure/autorest
changelog: ["@chronus/github/changelog", { repo: "Azure/autorest" }]

ignore:
- "@autorest/test-public-packages"

changeKinds:
breaking:
versionType: major
title: Breaking Changes
description: Changes that break existing features

feature:
versionType: minor
title: Features
description: New features added

fix:
versionType: patch
title: Bug Fixes
description: Fixes to existing features

deprecation:
versionType: minor
title: Deprecations
description: Deprecated features

internal:
versionType: none
title: Internal
description: Internal changes that are not user facing

changedFiles:
- "!**/*.md"
- "!**/*.test.ts"
2 changes: 1 addition & 1 deletion .default-eslintrc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ rules:
"require-atomic-updates": off

# Prettier config https://github.com/prettier/eslint-plugin-prettier#recommended-configuration
prettier/prettier: off # Don't run prettier in eslint we have rush check-format for that.
prettier/prettier: off # Don't run prettier in eslint we have pnpm format:check for that.
arrow-body-style: off
prefer-arrow-callback: off

Expand Down
4 changes: 3 additions & 1 deletion .github/workflows/consitency-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@ jobs:
uses: actions/setup-node@v4
with:
node-version: 22.x
- run: corepack enable
- run: pnpm install --frozen-lockfile
- run: |
LABEL_NAME=Publish
LABEL_URL=https://api.github.com/repos/Azure/autorest/issues/$PR_NUMBER/labels
Expand All @@ -25,7 +27,7 @@ jobs:
then
echo "Publish label was included in the PR, won't be checking for changelog."
else
npx @microsoft/rush change --verify || { echo 'If you run the rush publish command locally and meant to publish the changes, add the publish label to the pr.' ; exit 1; }
pnpm chronus verify || { echo 'If you meant to publish the changes, add the publish label to the pr.' ; exit 1; }
fi
env:
PR_NUMBER: ${{ github.event.pull_request.number }}
7 changes: 3 additions & 4 deletions .github/workflows/prepare-auto-release-branch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,10 @@ jobs:
with:
node-version: 22.x

- run: node common/scripts/install-run-rush.js install
- run: corepack enable

- run: pnpm install --frozen-lockfile
name: Install dependencies

- name: Create release branch
run: node ./eng/scripts/publish.mjs

- run: node common/scripts/install-run-rush.js update
name: Regen lock file
6 changes: 2 additions & 4 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -285,10 +285,8 @@ jspm_packages/
# Common toolchain intermediate files
temp

# Rush files
common/temp/**
package-deps.json
common/config/rush/shrinkwrap.yaml
# pnpm
.chronus/artifacts

# Code generation output for regression tests
core/test/regression
Expand Down
5 changes: 5 additions & 0 deletions .npmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
enable-pre-post-scripts=true
only-built-dependencies[]=tree-sitter
only-built-dependencies[]=tree-sitter-javascript
only-built-dependencies[]=tree-sitter-python
only-built-dependencies[]=tree-sitter-typescript
27 changes: 14 additions & 13 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ Then go to https://github.com/Azure/autorest/pull/new/publish/auto-release and c
- **On windows**: Make sure to choose to install the native build dependencies in the setup
- Alternatively follow instruction here https://github.com/nodejs/node-gyp
- `python` 3.x
- `pnpm` (enabled via corepack: `corepack enable`)

Optional recommendation:

Expand All @@ -21,25 +22,25 @@ Optional recommendation:

## First build

1. Install [rush.js](https://rushjs.io/pages/intro/get_started/) using
1. Enable corepack (if not already)

```bash
npm install -g @microsoft/rush
corepack enable
```

2. Install dependencies

```bash
rush update
pnpm install
```

3. Build

```bash
rush build
pnpm build

# or to do a force rebuild.
rush rebuild
pnpm rebuild
```

## Run in watch mode
Expand All @@ -48,9 +49,9 @@ When working on autorest it is recommended to have the compiler run in watch mod

```bash
# Run for all packages.
rush watch
pnpm watch
# Run for a specific package.
npm run watch
pnpm run watch
```

## Test
Expand All @@ -62,7 +63,7 @@ To run the test on the built product you have 2 options:
1. Run all the tests using

```bash
rush test:ci
pnpm test:ci
```

2. Run individual project tests(Recommended when working on test)
Expand All @@ -74,7 +75,7 @@ cd packages/<type>/<package>/
# Run test in interactive mode
npm test

# Alternatively you can run them once with coverage(Same as rush test:ci)
# Alternatively you can run them once with coverage(Same as pnpm test:ci)
npm run test:ci
```

Expand All @@ -84,7 +85,7 @@ npm run test:ci

```bash
# Run for all packages.
rush lint
pnpm lint
# Run for a specific package.
npm run lint
```
Expand All @@ -93,7 +94,7 @@ npm run lint

```bash
# Run for all packages.
rush clean
pnpm clean
# Run for a specific package.
npm run clean
```
Expand Down Expand Up @@ -124,10 +125,10 @@ node <path-to-repo>/packages/apps/autorest/entrypoints/app.js

Steps to do before making a pull request:

1. Run `rush change` and describe the change and if it should be a `major`, `minor` or `patch` version.
1. Run `pnpm change` and describe the change and if it should be a `major`, `minor` or `patch` version.

- `major`: If there is a breaking change.(Except `autorest`, `@autorest/core` and `@autorest/modelefour` packages which should use minor bump for that.)
- `minor`: If there is a new feature but not breaking(Except `autorest`, `@autorest/core` and `@autorest/modelefour` packages)
- `patch`: For any bug fix.

2. Run `rush format` to ensure the code is formatted correctly.
2. Run `pnpm format` to ensure the code is formatted correctly.

This file was deleted.

This file was deleted.

This file was deleted.

12 changes: 0 additions & 12 deletions common/config/rush/.npmrc

This file was deleted.

25 changes: 0 additions & 25 deletions common/config/rush/.npmrc-publish

This file was deleted.

Loading
Loading