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

Improve merging trace script #153

Merged
merged 3 commits into from
Aug 10, 2024
Merged

Conversation

rayandrew
Copy link
Collaborator

Close #148

Here I tried to run sed where it will

  • Remove all brackets
  • Remove empty lines
  • Add bracket at the beginning of file
  • Fix misleading destination file message to user

Here I tried to run `sed` where it will
- Remove all brackets
- Remove empty lines
- Add bracket at the beginning of file
- Fix misleading destination file message to user
@coveralls
Copy link

coveralls commented Aug 9, 2024

Coverage Status

coverage: 0.0%. remained the same
when pulling aec5c6a on feature/merge-trace
into f03b635 on develop.

Copy link
Member

@hariharan-devarajan hariharan-devarajan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good.

@hariharan-devarajan hariharan-devarajan merged commit 69ee384 into develop Aug 10, 2024
9 checks passed
@rayandrew rayandrew deleted the feature/merge-trace branch August 10, 2024 02:44
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.

Improve merge pfw script
3 participants