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

feat: createfile() and toFile() #23

Merged
merged 5 commits into from
Dec 14, 2023
Merged

Conversation

Bugs5382
Copy link
Owner

No description provided.

- initial code for file batch'ing creation
- simple unit test passes with a single MSH segment

#16

[ci skip]
- more unit tests added for different scenarios
- if you add an MSH to a file where there is already a BHS, it will add it the BHS and in order
- two more batches can be added to a file and each batch, as long as they are defined, can have their own message segments

#16

[ci skip]
- create a file
- can create a directory or place it where it needs to go recursive
- fix file header options
- update 2.7 specs with FHS
- update gitignore and npmignore for the temp folder used for file creation

#16

[ci skip]
@Bugs5382 Bugs5382 added this to the 1.0.0 milestone Dec 14, 2023
@Bugs5382 Bugs5382 self-assigned this Dec 14, 2023
- .toFile() method implemented on message and batch
- fixed an issue with value node and empty strings
- updated unit tests

#16

[ci skip]
@Bugs5382 Bugs5382 marked this pull request as ready for review December 14, 2023 02:21
@Bugs5382 Bugs5382 merged commit c38c4c5 into develop Dec 14, 2023
@Bugs5382 Bugs5382 deleted the feature/16-feat-method-tofile branch December 14, 2023 02:23
@Bugs5382
Copy link
Owner Author

🎉 This PR is included in version 1.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Bugs5382
Copy link
Owner Author

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant