Readthedocs.org: Remove `gold` app

Created on 25 Oct 2017  路  15Comments  路  Source: readthedocs/readthedocs.org

Currently, we have the gold app in the RTD codebase. We should break it out into a separate app, since it isn't relevant to other installs of RTD.

Accepted Improvement

All 15 comments

@ericholscher Please explain me how this could be implemented. I am new here and would like to contribute to it.

we have to do complete removal of 'gold' app?? @ericholscher
'gold' app lot of files, so it'll take time,finding all of thses files and removing it.

@ericholscher Hi, I want to take this issue and I was thinking of moving the gold app to the profiles app. Is it correct?

It sounds like it should be in it's own app, perhaps called gold. But I'm not sure.

@agjohnson what's the idea here? to move the gold app to the readthedocs-ext repo since it's not tied to RTD as software project?

@RichardLitt does it mean it needs a new repo?

Again, I'm not sure. Let's wait on one of the main maintainers to answer.

@RichardLitt whoa re the main maintainers?

@gaborbernat you can see it here https://docs.readthedocs.io/en/latest/team.html

@rtfd/core thoughts?

This isn't actionable by non-core folks, so not a good first issue.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Is this still valid?
I would like to work on this.
Also, what exactly is required here? Breaking gold app in multiple small apps or removing gold app and merging the code in other apps?

@dojutsu-user this is still valid, but only actionable for the core team (because the code needs to be moved to a private repo)

I think we should close it -- we're not going to remove gold any time soon, and likely won't put more work into it until we do the work to totally remove it from our codebase :)

Was this page helpful?
0 / 5 - 0 ratings