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

Added conspicuous messaging that this repo is no longer maintained. #149

Merged
merged 1 commit into from
Jul 13, 2015
Merged

Conversation

qbunt
Copy link

@qbunt qbunt commented Jul 12, 2015

With the thought that someone could actively assume that this repo/generator is being maintained and kept in 'battle-tested' state, I'm adding a message to the readme that shows it as such. I appreciate all the work that's been done here, but it's been stale for a really long time.

@eriktrom
Copy link
Contributor

I would be in favor of letting people know this is not under active development

@eastridge - this seems to be an accurate statement and rather than have people try it out only to find it outdated should we add the message that @qbunt is proposing?

@eriktrom
Copy link
Contributor

cc @kpdecker

@kpdecker
Copy link
Contributor

I don't have commit access to any Walmart projects anymore so I can't merge this. @jasonwebster is maintaining Thorax at this point so he would be a better person to discuss this with.

@qbunt
Copy link
Author

qbunt commented Jul 12, 2015

We can revise the wording or the formatting of that message, but I think everyone gets the concept here. We fell into this generator to find it outdated pretty late in our project, I'd like to avoid others doing the same.

@qbunt
Copy link
Author

qbunt commented Jul 13, 2015

@jasonwebster & @kpdecker let me know if I can help push this along.

eastridge added a commit that referenced this pull request Jul 13, 2015
Added conspicuous messaging that this repo is no longer maintained.
@eastridge eastridge merged commit 129eec8 into walmartlabs:master Jul 13, 2015
@eastridge
Copy link
Contributor

@qbunt thanks for the PR

@qbunt
Copy link
Author

qbunt commented Jul 13, 2015

Sure thing, thanks for accepting @eastridge, hope I didn't stomp on too many toes.

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

Successfully merging this pull request may close these issues.

4 participants