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

Add documentation and tooling for GOV.UK Components #444

Merged
merged 5 commits into from
Sep 2, 2014
Merged

Conversation

dsingleton
Copy link
Contributor

This aims to make it easier for developers to understand how components work, how they're structured and how to write their own.

It adds README guidance and a generator to create scaffolding for a new component.

I look forward to the inevitable pointing out of typos, which you are more than welcome to fix in place :]

@rboulton
Copy link
Contributor

As implicitly requested, fixed a few typos by pushing a commit.

Quite a lot of typos actually.

Looks good other than that; I'm happy to merge, but perhaps this wants review from, say, @edds, first.

@edds
Copy link
Contributor

edds commented Sep 1, 2014

Need to drop the metadata example as that has now been merged.

@rboulton
Copy link
Contributor

rboulton commented Sep 1, 2014

Also, I just noticed that the jenkins jobs for testing this PR have hung: "Build has been executing for 2 days 16 hr". I've cancelled them, but that needs investigation.

David Singleton and others added 5 commits September 2, 2014 11:44
Static is the authorative source for components, including documentation, so should be included here.
- As this grows it might want to be split into a README in a components specific directory, tho a good one doesn't exist yet
- It might be worth creating a generator to do the creation, just docing for now
Generates all the file scaffolding, in the right places, as
the files are a bit spread throughout static at the moment.

This will include all the files required to let the component
be demo'd on govuk-component

Appending to the json docs file is a bit shonky, but we're
not planning on using that as a docs method long term. It'll
likely be inline with template, or at least one doc file per
component, which will make this simpler.
Fixtures are a hash of hashes, the parent hash is fixture_name => fixture

The fixture is a hash of locals to be passed to the compontent view.
@dsingleton dsingleton force-pushed the new-component-docs branch 4 times, most recently from dc35623 to c9a56ef Compare September 2, 2014 11:49
@dsingleton
Copy link
Contributor Author

Cleaned up & rebased.

edds added a commit that referenced this pull request Sep 2, 2014
Add documentation and tooling for GOV.UK Components
@edds edds merged commit 56b4233 into master Sep 2, 2014
@edds edds deleted the new-component-docs branch September 2, 2014 12:23
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.

3 participants