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

Publish to NuGet + Documentation #49

Merged
merged 19 commits into from
Mar 12, 2025
Merged

Publish to NuGet + Documentation #49

merged 19 commits into from
Mar 12, 2025

Conversation

matejicko
Copy link
Collaborator

@matejicko matejicko commented Mar 10, 2025

This PR doesn't add any new functionality or features. Rather, I have checked whole repository, structure and overall experience for users and potential contributors.

During development of this PR, this repository became public. Recommended files CONTRIBUTING.md and CODE_OF_CONDUCT.md were added to support potential contributors of this project.

Pipeline update

Former pr-pipeline.yml was renamed to pipeline.yml and two new jobs were added:

  • publish-docs - each time current branch is merged to master, documentation web-site is updated and re-deployed on GitHub pages - Documentation
  • publish-nuget - when current branch is merged to master and version of the tool is increased, push to NuGet is performed

Both newly added jobs are dependent on success of build-and-test job.

Documentation

Former README.md used to represent user manual and documentation. All necessary information were spread to wiki/documentation of this project, are available on Documentation web-site, which is deployed on GitHub pages

Documentation web-site also includes specification of public API.

Publication

NuGet package TeaPie.Tool is already available. 🥳

@matejicko matejicko changed the title Publish to NuGet Publish to NuGet + Documentation Mar 11, 2025
When accessing variables **without specifying a level**, TeaPie searches **from the highest level to the lowest** and returns the **first match** found.

```csharp
if (!tp.ContainsVariable("IsDevlabEnvironment")) {
Copy link
Member

Choose a reason for hiding this comment

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

Možno len do budúcna issue. Vytvoriť metódu TrySetVariable, alebo SetVariebleIfNotDefined, alebo niečo také. Aby sa to nemuselo písať takto.
Ale to teraz isto nerob, len to zapíš.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Okay, good idea for the future

@matejicko matejicko added the unresolved comments PR with unresolved comments from reviewer(s). Label should be removed after resolution. label Mar 12, 2025
@matejicko matejicko merged commit fb9e2d0 into master Mar 12, 2025
4 checks passed
@matejicko matejicko deleted the publish branch March 12, 2025 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
unresolved comments PR with unresolved comments from reviewer(s). Label should be removed after resolution.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants