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

Update Readme #11

Merged
merged 3 commits into from
Mar 11, 2022
Merged

Update Readme #11

merged 3 commits into from
Mar 11, 2022

Conversation

radeksimko
Copy link
Member

@radeksimko radeksimko commented Mar 7, 2022

The aim is to answer likely FAQ and make the repo ready to become public.

It is best viewed rendered: https://github.com/hashicorp/syntax/blob/c18259a3fdfcf2478bf45c72894165050be67ee2/README.md

@radeksimko radeksimko requested a review from a team March 7, 2022 20:08
@radeksimko radeksimko marked this pull request as ready for review March 7, 2022 20:08
README.md Outdated Show resolved Hide resolved

## What editors this can be used with
Instead you can use the grammar through editors, editor extensions, or highlighting libraries which in turn may use this repository, for example:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Instead you can use the grammar through editors, editor extensions, or highlighting libraries which in turn may use this repository, for example:
In most cases you will use these TextMate grammars through editors, editor extensions, or highlighting libraries which in turn may use this repository, for example:

Copy link
Member Author

@radeksimko radeksimko Mar 8, 2022

Choose a reason for hiding this comment

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

I'm curious what are the other cases here, in which end-users would want to use the grammars directly? Maybe it wasn't clear what I meant by "end-user" above - should I clarify that to "Editor end-user"?

Copy link
Contributor

Choose a reason for hiding this comment

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

I think this can be resolved by splitting up this document as discussed in https://github.com/hashicorp/syntax/pull/11/files#r821403019


* HCL - TextMate
* Terraform - TextMate
Generally you should never need to interact with this repository directly.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Generally you should never need to interact with this repository directly.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved

We may consider distributing the grammars via a packaging system such as NPM in the future.

#### Grammar Mapping
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a great deep dive into how this all works, but it is a lot of info to drop in a README we don't expect to have external help just yet, which will probably change a lot in the next few cycles. I think this probably is better suited in the development.md or in a CONTRIBUTING.md.

Copy link
Member Author

@radeksimko radeksimko Mar 8, 2022

Choose a reason for hiding this comment

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

We could cut it down a bit and move some content into separate files but there are different types of content which we should not conflate and probably not put into CONTRIBUTING nor DEVELOPMENT.

From my POV there are different personas we're targeting here (1) end-users of editors/websites, (2) maintainers of extensions/websites (3) contributors to our grammars

  • (1) should generally never even need to know about this repo IMO, so we just send them away via links to extensions etc.
  • (2) they need to know how to use this but the instructions IMO shouldn't be in DEVELOPMENT or CONTRIBUTING, more like USAGE.md - as from their POV they're using it
  • (3) for these people we can put instructions into CONTRIBUTING.md (or development.md) as - these would be people contributing back to this repo e.g. to hcl.tmGrammar.json or terraform.tmGrammar.json.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not tied to filenames, but from your response we appear to be in agreement that this README should be a landing page and not a catalogue of all the different sets of knowledge required to use the files in this repo.

Extrapolating from your thoughts, I think we need:

  • README.md with sections briefly explaining the repo and the different types of uses cases. Each with links to get users/contributors where they need to go.
  • USAGE.md document outlining expected ways to consume these files
  • CONTRIBUTING.md document outlining what content we accept.

Since the goal of this PR is to get the project quickly to the point of unblocking other work, I'm fine with ticketing this as a follow up and merging this PR as is.

Copy link
Member Author

Choose a reason for hiding this comment

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

👍🏻 created #14 to track the remaining work

radeksimko and others added 2 commits March 8, 2022 07:55
@radeksimko radeksimko merged commit c7f7726 into main Mar 11, 2022
@radeksimko radeksimko deleted the update-readme branch March 11, 2022 12:31
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