Skip to content

feat(auth):CI PipeLine WorkFlow for Pullpiri[#106]#115

Merged
youngtaekiim merged 4 commits intoeclipse-pullpiri:refactoringfrom
akshaylg0314:refactoring
Jul 9, 2025
Merged

feat(auth):CI PipeLine WorkFlow for Pullpiri[#106]#115
youngtaekiim merged 4 commits intoeclipse-pullpiri:refactoringfrom
akshaylg0314:refactoring

Conversation

@akshaylg0314
Copy link
Contributor

@akshaylg0314 akshaylg0314 commented Jul 9, 2025

📝 PR Description
CI Pipeline Workflow for Pullpiri

🔗 Related Issue
Closes #106

🧪 Test Method
Github Action should run and check based on the Code change.

Below is the link for GitHub Action run for the pull request from the CI-Pipeline branch of my fork to the refactoring branch:
https://github.com/akshaylg0314/pullpiri/actions/runs/16491926519

Readme file:
Readme_ci_cd.txt
✅ Checklist
[✅] Code conventions are followed
[✅] Tests are added/modified
[✅] Documentation is updated (if necessary)

@akshaylg0314
Copy link
Contributor Author

Hey @youngtaekiim @leeeunkoo
The admin needs to enable the required status checks in the repository settings (as mentioned in the README) to block merges if checks fail.

Also, I’ve temporarily disabled the Markdown lint check for docs, as some adjustments are needed in the documentation structure before re-enabling it.

@basheerFZ
Copy link
Contributor

@edo.lee --> as admin you need to do this below , currently "No conflicts with base branch" is enabled by default, it does not wait for the checks to complete. once this is enabled , the option to merge will be enabled only after the checks are completed.

The admin needs to enable the required status checks in the repository settings (as mentioned in the README) to block merges if checks fail.

@youngtaekiim
Copy link
Contributor

Let's merge it as it is now and see how it is checked when registering a PR. Of course, we will have to block merge before verification is complete.

@youngtaekiim youngtaekiim merged commit 751d30e into eclipse-pullpiri:refactoring Jul 9, 2025
6 checks passed
@youngtaekiim youngtaekiim self-assigned this Jul 18, 2025
chaehee-lim pushed a commit to chaehee-lim/pullpiri that referenced this pull request Sep 3, 2025
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.

3 participants