Skip to content

Add documentation regarding new functionality of packit init - #1056

Merged
softwarefactory-project-zuul[bot] merged 1 commit into
packit:mainfrom
betulependule:packit-init-flag
Sep 15, 2025
Merged

Add documentation regarding new functionality of packit init#1056
softwarefactory-project-zuul[bot] merged 1 commit into
packit:mainfrom
betulependule:packit-init-flag

Conversation

@betulependule

@betulependule betulependule commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

Documentation now describes new requirements connected to the functionality of packit init that automatically adds a packit-specific pre-commit hook that validates .packit.yaml to the user's pre-commit onfiguration file (skipped if pre-commit configuration file is not present). Two new flags were also edit to modify this behavior:

--force-precommit : initializes an empty pre-commit onfiguration file if missing and adds aforementioned pre-commit hook
--without-precommit : skips this pre-commit-related behavior

Related to #2641

Merge at the same time as #2641

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule

Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule
betulependule requested review from majamassarini and removed request for majamassarini August 18, 2025 08:03
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule
betulependule marked this pull request as ready for review September 4, 2025 09:23
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule
betulependule requested a review from mfocko September 4, 2025 10:07

@mfocko mfocko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Except for the one typo, LGTM.

Comment thread docs/cli/init.md
Comment thread docs/cli/init.md Outdated
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@mfocko

mfocko commented Sep 15, 2025

Copy link
Copy Markdown
Member

pre-commit passed, but you still need to rebase @betulependule

Documentation now describes new requirements connected to the
functionality that automatically adds packit-specific pre-commit
check that validates .packit.yaml. Requirements are:

1. Working directory be a git repo
2. Pre-commit configuration file must be present

A new flag was also added, which causes packit init to automatically
create pre-commit configuration file in case it is missing.
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

@betulependule betulependule added the mergeit Merge via Zuul label Sep 15, 2025
@softwarefactory-project-zuul

Copy link
Copy Markdown
Contributor

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/cde169119b6e40ff98ba15ffc22e2e8f

✔️ pre-commit SUCCESS in 1m 14s

@softwarefactory-project-zuul
softwarefactory-project-zuul Bot merged commit a2d7054 into packit:main Sep 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants