Skip to content

Extension Development Improvements #1941

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 3 commits into from
May 7, 2018
Merged

Extension Development Improvements #1941

merged 3 commits into from
May 7, 2018

Conversation

WardenGnaw
Copy link
Member

  1. Updated tasks.json to 2.0
  2. Added tslint problem matcher
    • Weird issue that problem does not disappear when resolved.
  3. Adding tslint for class and function names.
    • class must be in PascalCase
    • funcitons must be in camelCase
  4. Addressing vulnerability warning

@WardenGnaw
Extension Development Improvements
7edbf0a 
1. Updated tasks.json to 2.0
2. Added tslint problem matcher
   - Weird issue that problem does not disappear when resolved.
3. Adding tslint for class and function names.
   - class must be in PascalCase
   - funcitons must be in camelCase
4. Addressing vulnerability warning
   - microsoft/vscode#49146
@WardenGnaw WardenGnaw self-assigned this May 5, 2018
@WardenGnaw WardenGnaw requested review from bobbrow and sean-mcmanus May 5, 2018 16:27
@WardenGnaw WardenGnaw merged commit ff31eef into microsoft:master May 7, 2018
@github-actions github-actions bot locked and limited conversation to collaborators Oct 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bobbrow bobbrow bobbrow approved these changes

@sean-mcmanus sean-mcmanus sean-mcmanus approved these changes

Assignees

@WardenGnaw WardenGnaw

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@WardenGnaw @bobbrow @sean-mcmanus