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

Port from C# to ghūl #62

Closed
wants to merge 4 commits into from
Closed

Port from C# to ghūl #62

wants to merge 4 commits into from

Conversation

degory
Copy link
Owner

@degory degory commented Feb 7, 2024

  • Re-implemented in ghūl
  • Renamed main pipe class from Pipe`1 to Pipe
  • Moved the Pipe class into a new Ghul.Pipes namespace

Notes:

  • Will require an update to the ghūl compiler to reference the new Ghul.Pipes.Pipe type before the | operator can be used with this version. (And actually, I'll probably take the opportunity to roll this functionality into the main ghūl runtime library package)

Bumps #major version

- Re-implemented in ghūl
- Renamed main pipe class from ``Pipe`1`` to `Pipe`
- Moved the `Pipe` class into a new `Ghul.Pipes` namespace

Notes:
- Will require an update to the ghūl compiler to reference the new `Ghul.Pipes.Pipe` type before the `|` operator can be used with this version. (And actually, I'll probably take the opportunity to roll this functionality into the main ghūl runtime library package)

Bumps #major version
- Fix wrong project folder for `dotnet pack`
- Fix wrong paths for packed NuGet package
- Add missing `dotnet tool restore` to CI/CD workflow
- Add missing logo image
- Fix wrong path for README.md
@degory
Copy link
Owner Author

degory commented Feb 7, 2024

Closing and moving to ghul-runtime, to avoid having to bootstrap over two changes at once.

@degory degory closed this Feb 7, 2024
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.

None yet

1 participant