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

Set Phpcs to use tags #8833

Merged
merged 1 commit into from
Oct 25, 2023
Merged

Set Phpcs to use tags #8833

merged 1 commit into from
Oct 25, 2023

Conversation

benmatselby
Copy link
Contributor

The project is moving to tags

  • I'm the package's author and/or maintainer.
  • I have have read [the docs][1].
  • I have tagged a release with a [semver][2] version number.
  • My package repo has a description and a README describing what it's for and how to use it.
  • My package doesn't add context menu entries. *
  • My package doesn't add key bindings. **
  • Any commands are available via the command palette.
  • Preferences and keybindings (if any) are listed in the menu and the command palette, and open in split view.
  • If my package is a syntax it doesn't also add a color scheme. ***
  • If my package is a syntax it is named after the language it supports (without suffixes like "syntax" or "highlighting").
  • I use [.gitattributes][3] to exclude files from the package: images, test files, sublime-project/workspace.

My package is already in the registry, I'm just updating it to use tags.

The project is moving to tags
Copy link
Collaborator

@packagecontrol-bot packagecontrol-bot left a comment

Choose a reason for hiding this comment

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

Automated testing result: SUCCESS

Packages modified:
  - Phpcs

@braver
Copy link
Collaborator

braver commented Oct 25, 2023

🙌🏻

@braver braver merged commit 6e8ea49 into wbond:master Oct 25, 2023
2 checks passed
@benmatselby benmatselby deleted the patch-1 branch October 27, 2023 16:20
@benmatselby
Copy link
Contributor Author

Hey there @braver

I've noticed that the package needs a review is that something I need to raise somewhere?

@braver
Copy link
Collaborator

braver commented Oct 29, 2023

@wbond needs to have a look at that.

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.

3 participants