Skip to content
This repository has been archived by the owner on Jan 29, 2021. It is now read-only.

3 - Repository prep structure #2

Merged
merged 1 commit into from
Jul 4, 2018

Conversation

cdcabrera
Copy link
Contributor

@cdcabrera cdcabrera commented Jul 2, 2018

Baseline commit in preparation for a documentation/locale level repository.

  • git and editor configuration
  • NPM resources
  • base ReadMe

Relates to research issue cloudigrade/frontigrade#25

updates #3

@cdcabrera cdcabrera requested review from a team July 2, 2018 20:56
@cdcabrera cdcabrera changed the title Repository prep structure 3 - Repository prep structure Jul 3, 2018
elyezer
elyezer previously approved these changes Jul 4, 2018
Copy link

@elyezer elyezer left a comment

Choose a reason for hiding this comment

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

LGTM, left a small comment that you may want to handle before merging.

README.md Outdated
@@ -0,0 +1,3 @@
# Wordigrade

Locale and documentation for Cloud Meter.
Copy link

Choose a reason for hiding this comment

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

s/Cloud Meter/Cloudigrade?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can update it since its the org name +1

* git and editor configuration
* NPM resources
* base ReadMe
@cdcabrera cdcabrera merged commit 7b3353d into cloudigrade:master Jul 4, 2018
@elyezer
Copy link

elyezer commented Jul 4, 2018

I am approving but I am not 100% sure if we should have Cloud Meter on the README, but anyway having Cloudigrade will make people aware of what this is related to

@cdcabrera
Copy link
Contributor Author

@elyezer gotcha, we use some of the Cloud Meter labeling inside Frontigrade. But agreed, we should probably bring it forward and look at cleaning it up

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants