[MNG-8165] Get rid of bashism creeped in by cstamas · Pull Request #1653 · apache/maven (original) (raw)
Navigation Menu
Appearance settings
- AI CODE CREATION
* GitHub CopilotWrite better code with AI
* GitHub Copilot appDirect agents from issue to merge
* MCP RegistryNewIntegrate external tools - DEVELOPER WORKFLOWS
* ActionsAutomate any workflow
* CodespacesInstant dev environments
* IssuesPlan and track work
* Code ReviewManage code changes - APPLICATION SECURITY
* GitHub Advanced SecurityFind and fix vulnerabilities
* Code securitySecure your code as you build
* Secret protectionStop leaks before they start - EXPLORE
* Why GitHub
* Documentation
* Blog
* Changelog
* Marketplace
- AI CODE CREATION
- BY COMPANY SIZE
* Enterprises
* Small and medium teams
* Startups
* Nonprofits - BY USE CASE
* App Modernization
* DevSecOps
* DevOps
* CI/CD
* View all use cases - BY INDUSTRY
* Healthcare
* Financial services
* Manufacturing
* Government
* View all industries
- BY COMPANY SIZE
- EXPLORE BY TOPIC
* AI
* Software Development
* DevOps
* Security
* View all topics - EXPLORE BY TYPE
* Customer stories
* Events & webinars
* Ebooks & reports
* Business insights
* GitHub Skills - SUPPORT & SERVICES
* Documentation
* Customer support
* Community forum
* Trust center
* Partners
- EXPLORE BY TOPIC
- COMMUNITY
* GitHub SponsorsFund open source developers - PROGRAMS
* Security Lab
* Maintainer Community
* Accelerator
* GitHub Stars
* Archive Program - REPOSITORIES
* Topics
* Trending
* Collections
- COMMUNITY
- Pricing
Provide feedback
We read every piece of feedback, and take your input very seriously.
Include my email address so I can be contacted
Saved searches
Use saved searches to filter your results more quickly
Appearance settings
Notifications You must be signed in to change notification settings
Additional navigation options
Merged
merged 1 commit into
Aug 14, 2024
ConversationCommits (1)ChecksFiles changed
Merged
[MNG-8165] Get rid of bashism creeped in#1653
merged 1 commit into
Conversation
cstamas commented
Copy link Copy Markdown
Member
Use of "==" is a bashism.
https://issues.apache.org/jira/browse/MNG-8165
`[[MNG-8165] Get rid of bashism creeped in](/apache/maven/pull/1653/commits/78735f7cbe8de0abe482c593d1f94501d10d5868 "[MNG-8165] Get rid of bashism creeped in
Use of "==" is a bashism.
https://issues.apache.org/jira/browse/MNG-8165") `
[78735f7](/apache/maven/pull/1653/commits/78735f7cbe8de0abe482c593d1f94501d10d5868)
Use of "==" is a bashism.
https://issues.apache.org/jira/browse/MNG-8165
cstamas requested review fromgnodet and michael-o
cstamas self-assigned this
cstamas added this to the 3.9.9 milestone
michael-o approved these changes Aug 14, 2024
michael-o left a comment
Copy link Copy Markdown
Member
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM krb5/krb5@fefd465
slawekjaranowski approved these changes Aug 14, 2024
cstamas merged commit 7185b0a into apache:maven-3.9.x
cstamas deleted the maven-3.9.x-MNG-8165 branch
jira-importer commented
Copy link Copy Markdown
Resolve #9687
mend-for-github-com Bot mentioned this pull request
Update Mend: high confidence minor and patch dependency updatessnowdensb/wildfly#141
Open
olamy mentioned this pull request
3.10.x Upgrade slf4j to 2.0.17#11880
Merged
8 tasks
mend-for-github-com Bot mentioned this pull request
Update Mend: high confidence minor and patch dependency updates - autoclosedjgeraigery/S3Mock#3
Closed
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})
Sign up for free to join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
michael-o michael-o approved these changes
slawekjaranowski slawekjaranowski approved these changes
gnodet Awaiting requested review from gnodet
Assignees
Labels
None yet
Projects
None yet
Milestone
Development
Successfully merging this pull request may close these issues.