-
Notifications
You must be signed in to change notification settings - Fork 26
Suggestion: improve readme to help people reuse this #12
Comments
I believe most of these will be taken care of by #6, which I will theoretically get time to go back to someday. |
Pushed small edits to existing readme to address remaining asks in this issue. Otherwise, resolved. |
@psswf1172 Hmm, I think something went wrong when you were trying to submit them...the only commit I'm seeing of yours is https://github.com/psswf1172/18f-glossary/commit/51b1b3c4747ec8844ac09238c75abbceaf4d98f3. Would you mind deleting your fork (from the settings page), re-forking, and trying to commit and send a pull request again? |
@afeld Is this issue still current? I see the check marks but couldn't tell based on the chatter. |
I think the last checkbox is still relevant, yeah. |
Now that #32 is merged, should this issue be closed as well? @afeld @mvattuone |
It'd be nice to add some basic info to this readme since people inside and outside 18F may be interested in reusing a glossary widget for future projects. Suggestions:
The text was updated successfully, but these errors were encountered: