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

Logging framework - chapter outline #56

Merged
merged 4 commits into from
Apr 21, 2024
Merged

Conversation

noelrivasc
Copy link
Contributor

Description

This PR is a draft outline for the topic proposed in issue #13 : the logging framework.

Current state

The aim at this point is to present an outline that is enough to judge if the approach is adequate, and if the scope is sufficient for a chapter.

Under /iterations, the first two are drafted. The idea is for each of them to build on the previous one, improving the design while offering opportunities to demonstrate language features and use.

A note on the use of platforms

In #13, the latest suggestion is to write the logging framework using two platforms (CLI and web server) to compare and contrast, and show how the platform influences design.

The outline proposed only includes the CLI implementation. My reasoning is that the differences between the web server and cli platforms aren't big enough that we end up with very different designs. The cli implementation on its own brings good opportunities to highlight the relationship between Roc and the underlying platform, and different ways to handle that relationship.

@noelrivasc noelrivasc changed the title Logging framework - Chapter outline Logging framework - chapter outline Apr 15, 2024
@gvwilson
Copy link
Collaborator

I realize this is a work in progress, but may I merge it as-is so I have material when I start rearranging order?

@noelrivasc
Copy link
Contributor Author

I realize this is a work in progress, but may I merge it as-is so I have material when I start rearranging order?

Yes, that's perfect.

@gvwilson gvwilson merged commit 6b00502 into roc-lang:main Apr 21, 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.

2 participants