+
Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

release: 0.8.0 #2925

Merged
merged 10 commits into from
Aug 2, 2022
Merged

release: 0.8.0 #2925

merged 10 commits into from
Aug 2, 2022

Conversation

ematipico
Copy link
Contributor

Summary

Closes #2835

Test Plan

@ematipico ematipico changed the title chore: update documentation release: 0.8.0 Jul 25, 2022
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jul 25, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9af2c4a
Status:⚡️  Build in progress...

View logs


> Default: `80`

#### `javascript.formatter.quoteStyle`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

globals have been ignored on purpose because we don't use it yet

@ematipico ematipico marked this pull request as ready for review July 27, 2022 07:29
@ematipico ematipico requested a review from a team July 27, 2022 07:29
Copy link
Contributor

@leops leops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that this release is moving forward with state of the linter, should we add Linters to the categories of the VSCode extension ?
Also if the goal of this PR is to kick off the release on merge, I think it needs to bump the version numbers of the LSP and CLI in order to trigger the corresponding workflows once the changes land on main


#### `linter.rules.js.recommended`

Enables the recommended rules for the category `JavaScript`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a page we can link through so that people can see the recommended rules?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately not for the categories themselves

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a link to the list of rules

@ematipico ematipico merged commit a62dd0a into main Aug 2, 2022
@ematipico ematipico deleted the release/0.8.0 branch August 2, 2022 16:09
IWANABETHATGUY pushed a commit to IWANABETHATGUY/tools that referenced this pull request Aug 22, 2022
Co-authored-by: Micha Reiser <micha@rome.tools>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

☂️ Documentation changes 0.8.0
3 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载