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

[Docs] Add CONTRIBUTING.md #2408

Merged
merged 1 commit into from
Nov 27, 2024
Merged

[Docs] Add CONTRIBUTING.md #2408

merged 1 commit into from
Nov 27, 2024

Conversation

Kocal
Copy link
Member

@Kocal Kocal commented Nov 27, 2024

Q A
Bug fix? no
New feature? no
Issues Fix #...
License MIT

Contributing was never easy for newcomers, let's simplify that a lot :) cc @smnandre

Rendered: https://github.com/Kocal/symfony-ux/blob/doc-contributing/CONTRIBUTING.md

@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Nov 27, 2024
@Kocal Kocal changed the title Add CONTRIBUTING.md [Docs] Add CONTRIBUTING.md Nov 27, 2024
@Kocal Kocal added the docs Improvements or additions to documentation label Nov 27, 2024
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated
Comment on lines 79 to 85
To help you with assets, we provide the following commands that can be run at the root of the repository or in each package directory:
- `yarn run build`: build (compile) assets,
- `yarn run test`: run the tests (through Vitest),
- `yarn run lint`: lint assets (through Biome.js),
- `yarn run format`: format assets (through Biome.js),
- `yarn run check-lint`: check assets linting (through Biome.js),
- `yarn run check-format`: check assets formatting (through Biome.js)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
To help you with assets, we provide the following commands that can be run at the root of the repository or in each package directory:
- `yarn run build`: build (compile) assets,
- `yarn run test`: run the tests (through Vitest),
- `yarn run lint`: lint assets (through Biome.js),
- `yarn run format`: format assets (through Biome.js),
- `yarn run check-lint`: check assets linting (through Biome.js),
- `yarn run check-format`: check assets formatting (through Biome.js)
To help you with assets, we provide the following commands to lint, build, and test assets :
- `yarn run build`: build (compile) assets,
- `yarn run lint`: lint assets
- `yarn run format`: format assets
- `yarn run test`: run the tests

You can run the following commands either at the root of the repository or within each package directory.

Copy link
Member

Choose a reason for hiding this comment

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

i'd say "Let's focus on minimum knowledge / info "

Copy link
Member Author

@Kocal Kocal Nov 27, 2024

Choose a reason for hiding this comment

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

Yes, but commands behave differently depending of where you run them

$ git checkout my-feature-branch && \
git rebase upstream/2.x && \
git push -u origin my-feature-branch
```
Copy link
Member

Choose a reason for hiding this comment

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

Just a note for late: what about addind gh cli usage ?

gh repo sync

Copy link
Member Author

@Kocal Kocal Nov 27, 2024

Choose a reason for hiding this comment

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

It was one of my first attentions, but... 🤷🏻

TIL, I always use gco 2.x && gf upstream && grb upstream/2.x && git push 😅

Copy link
Member

Choose a reason for hiding this comment

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

sfux 😆

Copy link
Member

@smnandre smnandre left a comment

Choose a reason for hiding this comment

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

Amazing start. Thank you very much @Kocal.

@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Nov 27, 2024
@Kocal Kocal merged commit dadc81b into symfony:2.x Nov 27, 2024
55 of 56 checks passed
@Kocal Kocal deleted the doc-contributing branch November 27, 2024 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation Status: Reviewed Has been reviewed by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants