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

Converter to go from TAML to YAML #42 #81

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

cgleadall
Copy link

@cgleadall cgleadall commented Nov 23, 2020

Here is a simple StringBuilder implementation of TAML to YAML.
I would like some help to resolve the merge conflicts, as my fork was taken before the most recent changes. Can I merge across repos?

Changes:
Added 3 projects to a folder called: Yaml.Writer.Taml.NET

  1. Yaml.Writer.Taml.NET - the actual converter
  2. Test.Yaml.Writer.Taml.NET - the unit test project
  3. Yaml.Writer.Cli - a simple CLI to read a TAML file and Console.WriteLine the YAML equivalent.

Please let me know what I can do next.

@csharpfritz
Copy link
Owner

This looks interesting.. I'm taking a look today on stream. Perhaps this becomes part of a larger TAML Command-line utility app

@csharpfritz
Copy link
Owner

@cgleadall - can you merge the current solution file, and let's set this up as a starting point for a TamlCli that performs the various documentation format shifting and document manipulation

@csharpfritz
Copy link
Owner

Last item: let's get this unit test project included in the GitHub action for .NET code

@cgleadall
Copy link
Author

@csharpfritz I have made a mess of my history pulling in the latest from your repo :)
Both of your comments above are addressed in this PR now:

  • latest main is included
  • The unit tests got picked up automatically in the GitHub Actions.

As for the TAML CLI - I see issue #69

@cgleadall cgleadall marked this pull request as ready for review January 16, 2021 23:41
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.

2 participants