Skip to content

feat: add docs for build in-obs#897

Merged
softwarefactory-project-zuul[bot] merged 3 commits intopackit:mainfrom
rxbryan:obs-support
Feb 1, 2025
Merged

feat: add docs for build in-obs#897
softwarefactory-project-zuul[bot] merged 3 commits intopackit:mainfrom
rxbryan:obs-support

Conversation

@rxbryan
Copy link
Contributor

@rxbryan rxbryan commented Jun 26, 2024

Add docs for packit build in-obs

@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul
Copy link
Contributor

@softwarefactory-project-zuul
Copy link
Contributor


* SPEC file for the project placed in the repository.
* Packit config file placed in the repository.

Copy link

Choose a reason for hiding this comment

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

We definitely have to mention here that the user must have osc configured. Otherwise nothing in here will work.

Copy link
Member

Choose a reason for hiding this comment

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

I've added it.

@softwarefactory-project-zuul
Copy link
Contributor

$ packit build in-obs --project <project_name>
```

## Help
Copy link
Member

Choose a reason for hiding this comment

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

I am not sure if you know this, but this is being regenerated automatically from the real help message so you don't need to take care of updating.. Also, what you put into the help message can be edited in one place (~ packit project) and is then visible both in the help message and on docs...

Copy link
Member

Choose a reason for hiding this comment

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

The script is here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay so the help section is updated automatically

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can we leave the other sections in?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, yes!

@softwarefactory-project-zuul
Copy link
Contributor

@rxbryan rxbryan force-pushed the obs-support branch 2 times, most recently from fb3eb7c to e2e1628 Compare July 24, 2024 14:39
@softwarefactory-project-zuul
Copy link
Contributor

@lachmanfrantisek
Copy link
Member

@rxbryan can you please rebase this pull-request? (Or, should I take care of it?)

@rxbryan rxbryan marked this pull request as ready for review September 4, 2024 18:31
@softwarefactory-project-zuul
Copy link
Contributor

@rxbryan
Copy link
Contributor Author

rxbryan commented Sep 4, 2024

@rxbryan can you please rebase this pull-request? (Or, should I take care of it?)

Done.

I have also marked the PR ready for review

rxbryan and others added 2 commits February 1, 2025 10:49
@softwarefactory-project-zuul
Copy link
Contributor

@lachmanfrantisek
Copy link
Member

I've rebased the PR and resolved the missing bits.

@softwarefactory-project-zuul
Copy link
Contributor

Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@softwarefactory-project-zuul
Copy link
Contributor

@lachmanfrantisek lachmanfrantisek added the mergeit Merge via Zuul label Feb 1, 2025
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ pre-commit SUCCESS in 1m 03s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit cc9595a into packit:main Feb 1, 2025
1 check 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.

3 participants