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

Remove api extractor #854

Merged
merged 6 commits into from
Aug 6, 2021
Merged

Conversation

rurikoaraki
Copy link
Collaborator

Platforms Impacted

  • iOS
  • macOS
  • win32 (Office)
  • windows
  • android

Description of changes

Remove api-extractor from repo. The package doesn't seem to be used, so we should rip it out for now.

This change includes removing the relevant script tasks from each package.json, as well as the api-extractor files from each package. The generated api file in theme-ramp has been deleted since it's probably out of date and the package is deprecated anyway.

Verification

Build succeeds

Before After
Screenshot or description before this change Screenshot or description with this change

Pull request checklist

This PR has considered (when applicable):

  • Automated Tests
  • Documentation and examples
  • Keyboard Accessibility
  • Voiceover
  • Internationalization and Right-to-left Layouts

@rurikoaraki rurikoaraki merged commit 56ccce4 into microsoft:master Aug 6, 2021
@rurikoaraki rurikoaraki deleted the remove_api_extractor branch January 6, 2022 21:03
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