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

Merge to master #19

Merged
merged 8 commits into from
Sep 12, 2024
Merged

Merge to master #19

merged 8 commits into from
Sep 12, 2024

Conversation

hootanht
Copy link
Owner

No description provided.

Update the CD workflow to run on pull requests and ensure the deploy job runs only on push to the master branch.

* Add `pull_request` trigger to the `on` section in `.github/workflows/CD.yml` to run the CD workflow on pull requests.
* Ensure the `deploy` job in `.github/workflows/CD.yml` runs only when a commit is pushed to the `master` branch.
* Change CI badge links in the Persian Date Public Version table to use `CI.yml`
* Update CI pipeline description to reference `.github/workflows/CI.yml` instead of `.github/workflows/CD.yml`
Fix the CI name to CI and ensure CD runs on pull request
…irements

Add setup and run instructions to `README.md`.

* **Requirements**: Add a section mentioning the .NET 8.0 SDK requirement.
* **Setup and Run**: Add a section with steps to install .NET 8.0 SDK, restore dependencies, build the project, and run tests.
Improve readme file and add how to setup and run project and the requirements
Add GitHub Packages registry configuration and update CD pipeline to publish package to GitHub Packages.

* **PersianDate/PersianDate.csproj**
  - Update `<RepositoryUrl>` to `https://github.com/hootanht/PersianDate`.
  - Change `<PackageSource>` to `https://nuget.pkg.github.com/hootanht/index.json`.

* **.github/workflows/CD.yml**
  - Add step to authenticate with GitHub Packages using the `GITHUB_TOKEN` secret.
  - Add step to publish NuGet package to GitHub Packages.
  - Rename existing step to publish NuGet package to NuGet.org.
@hootanht hootanht merged commit d27e247 into master Sep 12, 2024
3 checks passed
hootanht added a commit that referenced this pull request Sep 12, 2024
Merge pull request #19 from hootanht/develop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant