Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: escape dollar symbol in docker-compose.xml for linux #721

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

lizhanhui
Copy link
Contributor

fix #720

Signed-off-by: Li Zhanhui <lizhanhui@gmail.com>
Copy link

codecov bot commented Nov 24, 2023

Codecov Report

Merging #721 (83720e2) into main (58e24aa) will increase coverage by 0.02%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #721      +/-   ##
============================================
+ Coverage     57.42%   57.45%   +0.02%     
- Complexity     1221     1224       +3     
============================================
  Files           131      131              
  Lines          8029     8029              
  Branches        736      736              
============================================
+ Hits           4611     4613       +2     
  Misses         3020     3020              
+ Partials        398      396       -2     
Components Coverage Δ
RocketMQ Broker ∅ <ø> (∅)
RocketMQ Common 5.80% <ø> (ø)
RocketMQ Controller 54.88% <ø> (ø)
RocketMQ Proxy 42.75% <ø> (+0.12%) ⬆️
RocketMQ Store 76.98% <ø> (ø)
RocketMQ Stream ∅ <ø> (∅)
RocketMQ Metadata 35.29% <ø> (ø)

see 1 file with indirect coverage changes

@lizhanhui lizhanhui requested a review from daniel-y November 24, 2023 06:58
@ShadowySpirits ShadowySpirits merged commit f2c1846 into main Nov 24, 2023
13 checks passed
@ShadowySpirits ShadowySpirits deleted the escape_compose branch November 24, 2023 07:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: '$' requires escape for on Linux for docker-compose.yaml
2 participants