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.

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

[TypeSpecValidation] Implement as ESLint Rules #31909

Draft
wants to merge 59 commits into
base: main
Choose a base branch
from
Draft

[TypeSpecValidation] Implement as ESLint Rules #31909

wants to merge 59 commits into from

Conversation

mikeharder
Copy link
Member

No description provided.

@openapi-pipeline-app openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Dec 17, 2024 •
edited
Loading

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Protected Files has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide

@openapi-pipeline-app openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Dec 17, 2024 •
edited
Loading

PR validation pipeline restarted successfully. If there is ApiView generated, it will be updated in this comment.

mikeharder and others added 30 commits December 19, 2024 01:39
@mikeharder
Add code coverage
8978c86
@mikeharder
Normalize paths so tests pass on win32
83d874c
@mikeharder
normalize expected value
786140b
@mikeharder
Rename job
40c0796
@mikeharder
Add comment
908df37
@mikeharder
Always return absolute path
5fe0a1f
@mikeharder
Resolve path before splitting
3009311
@mikeharder
Merge branch 'main' into eslint-plugin-tsv
d130711
@mikeharder
Add stub for e2e vite tests
a92698f
@mikeharder
Remove comment
3e93efa
@mikeharder
Add ESLint with names
41814f9
@mikeharder
Add dummy test to prevent errors
5907a21
@mikeharder
Stronger config typing
d0ae1de
@mikeharder
Move e2e test
49d09b6
@mikeharder
Run tests when contoso changes
2563cd2
@mikeharder
Add e2e test using real filesystem
6a555fb
@mikeharder
Remove old test
dd472f8
@mikeharder
Improve test name
d7439da
@mikeharder
Fix e2e tests
90fab86
@mikeharder
Exclude interface from coverage
9f5ad9b
@mikeharder
Exclude interfaces from coverage
fac51d7
@mikeharder
Add Management e2e test
313f926
@mikeharder
improve error validation
5cb8558
@mikeharder
Handle invalid path errors
29c9f1f
@mikeharder
Merge branch 'main' into eslint-plugin-tsv
b02de1e
@mikeharder
Remove e2e scripts
81f2cc8
@mikeharder
Merge branch 'main' into eslint-plugin-tsv
3a0425a
@mikeharder
Remove problematic vitest config
c7493c1
@mikeharder
Revert "Remove problematic vitest config"
adbc241 
This reverts commit c7493c1.
@mikeharder
Exclude vitest config from compilation
61fe81b 
- Prevents confusion caused if vite finds multiple config files (JS and TS)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@weshaggard weshaggard Awaiting requested review from weshaggard weshaggard will be requested when the pull request is marked ready for review weshaggard is a code owner

@benbp benbp Awaiting requested review from benbp benbp will be requested when the pull request is marked ready for review benbp is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mikeharder