chore: Add 10UP Deploy #2469
chore: Add 10UP Deploy #2469
Conversation
shohag121
added
Needs: Dev Review
WalkthroughThis pull request introduces two significant changes: the addition of a Changes
Assessment against linked issues
Suggested labels
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Outside diff range and nitpick comments (3)
.github/workflows/deploy.yml (3)
3-6: Consider restricting tag pattern to semantic versioningThe current configuration triggers on any tag. Consider restricting it to semantic versioning pattern to prevent accidental deployments.
on: push: tags: - - "*" + - "v[0-9]+.[0-9]+.[0-9]+*"
20-24: Optimize npm installation and build processConsider adding npm caching and version pinning for faster and more reliable builds.
+ - name: Setup Node.js + uses: actions/setup-node@v4 + with: + node-version: '18' + cache: 'npm' - name: Build run: | composer install --no-dev - npm install + npm ci npm run build
1-50: Consider adding post-deployment verification and notificationsTo enhance the deployment workflow, consider:
- Adding a post-deployment verification step to ensure the plugin is accessible on WordPress.org
- Implementing Slack/Discord notifications for deployment status
- Adding timeout configurations for long-running steps
- Implementing retry logic for potentially flaky operations
Would you like assistance in implementing any of these improvements?
🧰 Tools
🪛 actionlint (1.7.4)
13-13: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
46-46: the runner of "softprops/action-gh-release@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (23)
.wordpress-org/banner-1544x500.pngis excluded by!**/*.png.wordpress-org/banner-772x250.pngis excluded by!**/*.png.wordpress-org/icon-128x128.gifis excluded by!**/*.gif.wordpress-org/icon-256x256.gifis excluded by!**/*.gif.wordpress-org/screenshot-1.pngis excluded by!**/*.png.wordpress-org/screenshot-10.pngis excluded by!**/*.png.wordpress-org/screenshot-11.pngis excluded by!**/*.png.wordpress-org/screenshot-12.pngis excluded by!**/*.png.wordpress-org/screenshot-13.pngis excluded by!**/*.png.wordpress-org/screenshot-14.pngis excluded by!**/*.png.wordpress-org/screenshot-15.pngis excluded by!**/*.png.wordpress-org/screenshot-16.pngis excluded by!**/*.png.wordpress-org/screenshot-17.pngis excluded by!**/*.png.wordpress-org/screenshot-18.pngis excluded by!**/*.png.wordpress-org/screenshot-19.pngis excluded by!**/*.png.wordpress-org/screenshot-2.pngis excluded by!**/*.png.wordpress-org/screenshot-3.pngis excluded by!**/*.png.wordpress-org/screenshot-4.pngis excluded by!**/*.png.wordpress-org/screenshot-5.pngis excluded by!**/*.png.wordpress-org/screenshot-6.pngis excluded by!**/*.png.wordpress-org/screenshot-7.pngis excluded by!**/*.png.wordpress-org/screenshot-8.pngis excluded by!**/*.png.wordpress-org/screenshot-9.pngis excluded by!**/*.png
📒 Files selected for processing (2)
.distignore(1 hunks).github/workflows/deploy.yml(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- .distignore
🧰 Additional context used
🪛 actionlint (1.7.4)
.github/workflows/deploy.yml
13-13: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
46-46: the runner of "softprops/action-gh-release@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
🔇 Additional comments (3)
.github/workflows/deploy.yml (3)
25-28: LGTM!
The SVN installation steps are properly implemented with update and install commands.
38-44: LGTM!
The asset update configuration is properly implemented with appropriate settings.
29-37: LGTM! Verify secrets configuration
The deployment configuration looks good with proper secret handling.
| - name: Checkout code | ||
| uses: actions/checkout@v2 |
There was a problem hiding this comment.
Update checkout action to latest version
The actions/checkout@v2 is outdated. Update to the latest stable version to ensure security and compatibility.
- name: Checkout code
- uses: actions/checkout@v2
+ uses: actions/checkout@v4📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - name: Checkout code | |
| uses: actions/checkout@v2 | |
| - name: Checkout code | |
| uses: actions/checkout@v4 |
🧰 Tools
🪛 actionlint (1.7.4)
13-13: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
| - name: Create GitHub release | ||
| uses: softprops/action-gh-release@v1 | ||
| with: | ||
| files: ${{github.workspace}}/${{ github.event.repository.name }}.zip | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
There was a problem hiding this comment.
Update GitHub release action to latest version
The softprops/action-gh-release@v1 is outdated. Update to the latest stable version.
- name: Create GitHub release
- uses: softprops/action-gh-release@v1
+ uses: softprops/action-gh-release@v2
with:
files: ${{github.workspace}}/${{ github.event.repository.name }}.zip
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}Committable suggestion skipped: line range outside the PR's diff.
🧰 Tools
🪛 actionlint (1.7.4)
46-46: the runner of "softprops/action-gh-release@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue
(action)
| - name: Setup PHP 7.4 | ||
| uses: shivammathur/setup-php@v2 | ||
| with: | ||
| php-version: '7.4' | ||
| coverage: none | ||
| tools: composer:v2 |
There was a problem hiding this comment.
Update PHP version to supported release
PHP 7.4 has reached end of life. Consider updating to a supported PHP version (8.1 or 8.2) for security and performance benefits.
- name: Setup PHP 7.4
uses: shivammathur/setup-php@v2
with:
- php-version: '7.4'
+ php-version: '8.1'
coverage: none
tools: composer:v2📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - name: Setup PHP 7.4 | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: '7.4' | |
| coverage: none | |
| tools: composer:v2 | |
| - name: Setup PHP 7.4 | |
| uses: shivammathur/setup-php@v2 | |
| with: | |
| php-version: '8.1' | |
| coverage: none | |
| tools: composer:v2 |
mrabbani
added
Needs: Author Reply
and removed
Needs: Dev Review
|
@mrabbani vai, I have already given a task to marketing team to provide me some updated image of dokan plugin for wp org few days back and probably they will provide the images to me on next week. If this 10UP related deployment dependency is very urgent then kindly proceed onto that for now. Once the updated screenshot images will be available to me I will provide those resources at that time and on following deployment images can be updated as well. |
All Submissions:
Changes proposed in this Pull Request:
Related Pull Request(s)
Closes
How to test the changes in this Pull Request:
Changelog entry
Title
Detailed Description of the pull request. What was previous behaviour
and what will be changed in this PR.
Before Changes
Describe the issue before changes with screenshots(s).
After Changes
Describe the issue after changes with screenshot(s).
Feature Video (optional)
Link of detailed video if this PR is for a feature.
PR Self Review Checklist:
FOR PR REVIEWER ONLY:
Summary by CodeRabbit
.distignorefile to streamline the distribution process by excluding unnecessary files and directories.These enhancements improve the efficiency and organization of the plugin's deployment process.