Skip to content

Remove watchForChanges #2324

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 24, 2023
Merged

Remove watchForChanges #2324

merged 1 commit into from
Aug 24, 2023

Conversation

@tjprescott
Copy link
Member

Fix #2264.

@github-actions
Copy link
Contributor

Changes in this PR will be published to the following url to try(check status of TypeSpec Pull Request Try It pipeline for publish status):
Playground: https://cadlplayground.z22.web.core.windows.net/prs/2324/

Website: https://cadlwebsite.z1.web.core.windows.net/prs/2324/

@tjprescott tjprescott merged commit 44cad71 into microsoft:main Aug 24, 2023
@tjprescott tjprescott deleted the removeWatchForChanges branch August 24, 2023 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bterlson bterlson bterlson approved these changes

@daviwil daviwil Awaiting requested review from daviwil

@markcowl markcowl Awaiting requested review from markcowl markcowl is a code owner

@allenjzhang allenjzhang Awaiting requested review from allenjzhang

@timotheeguerin timotheeguerin Awaiting requested review from timotheeguerin timotheeguerin is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deprecate/remove watchForChanges compiler options

2 participants

@tjprescott @bterlson