Wyatt-FOSS/wyatt-packages
Wyatt-FOSS/wyatt-packages
6
2
Fork 0
Code Issues 20 Pull Requests 3 Actions Packages Projects Releases Activity

Add pre-commit rules #72

New Issue
Open
opened 2022-12-07 23:46:51 +00:00 by hugo · 2 comments
hugo commented 2022-12-07 23:46:51 +00:00
Owner
Copy Link

Maybe we can add pre-commit at the root to:

Maybe we can add [pre-commit](https://pre-commit.com/) at the root to: - dart format ? - dart analyze ? - sort imports ? (with <https://github.com/fluttercommunity/import_sorter>) - validate depedencies ? (with <https://pub.dev/packages/dependency_validator>) - check commit message ? (with <https://github.com/compilerla/conventional-pre-commit>)
hugo self-assigned this 2022-12-07 23:46:51 +00:00
hugo commented 2022-12-13 21:18:57 +00:00
Author
Owner
Copy Link

Add pre-commit instruction in readme

Add pre-commit instruction in readme
hugo commented 2023-02-07 13:13:01 +00:00
Author
Owner
Copy Link

In my opinion it is only useful to force the conventional commit. The analyzer/linter should not be forced in dev, and could even be a task to be checked by ci/cd for example.

In any case, pre-commit is a tool that everyone has the choice to activate or not on their machine. We only share some config.

In my opinion it is only useful to force the conventional commit. The analyzer/linter should not be forced in dev, and could even be a task to be checked by ci/cd for example. In any case, pre-commit is a tool that everyone has the choice to activate or **not** on their machine. We only share some config.
hugo added the
priority
low
config
documentation
proposal
 labels 2023-04-07 12:02:58 +00:00
hugo changed title from Proposal(packages): add pre-commit rules to Add pre-commit rules 2023-04-07 12:16:42 +00:00
hugo added the
package
all
 label 2023-04-07 12:18:03 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
package
all
Related to all package
package
analysis
package
architecture
package
authentication
package
bloc_helper
package
bloc_layout
package
component_copy_with
package
crud
package
form
package
http
package
i18n
package
notification
package
proposal
New package proposal
package
type_utils
package
ui_components
package
ui_kit
package
ui_layout
assets
Related to assets
breaking
Breaking changes.
bug
Unexpected problem or unintended behavior
build
Related to build
config
Related to configuration
dependencies
Update dependencies
documentation
Improvements or additions to documentation
duplicate
This issue or pull request already exists
enhancement
New feature or request
invalid
This doesn't seem right
logic
Related to business logic.
needs review
Needs review
next
In next major release/milestone
priority
critical
The priority is critical
priority
high
The priority is high
priority
low
The priority is low
priority
medium
The priority is medium
proposal
Proposal for new feature or request. Need discussion.
question
Further information is requested
refactor
Refactoring code
run
Related to runtime.
test
Related to tests
ui
Related to user interface.
wontfix
This will not be worked on
No Label
package
all
config
documentation
priority
low
proposal
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
alex angele.perraudin hugo malo mathieu.dejeans parfait.tapsoba.007
No Assignees hugo
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Wyatt-FOSS/wyatt-packages#72
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?