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

(site) add site from ned-kelly/libretime-website #584

Closed
wants to merge 1 commit into from

Conversation

frecuencialibre
Copy link
Contributor

@hairmare
Copy link
Member

hairmare commented Nov 4, 2018

I have a branch prepared at master...radiorabe:chore/integrate-site that preserves @ned-kelly's edit history. I think we should onboard him as a Maintainer as well so he can continue maintaining the page.

I also didn't find time to integrate his .travis.yml with the main one, it will be fairly straight forward though.

@frecuencialibre
Copy link
Contributor Author

frecuencialibre commented Nov 5, 2018

I have a branch prepared

sweet!

onboard him as a Maintainer

big +1 to adding @ned-kelly as a maintainer. @Robbt cómo ves?

i know i haven't been involved in the project for nearly as long as others, but will share my opinion here anyway :). I'd love to arrive at a consensus that reduces dependence on you Lucas for non-RelEng stuff since i think we could definitely move faster as a project.

I think we could better distribute tasks between maintainers without negatively impacting the quality of the product IF we had well-understood consensus about responsibility for specific tasks. Things would be accomplished differently (eg. for the site i'd probably just commit processed css files instead of researching how to get node into travis), but hopefully the final product wouldn't suffer, and input from all would still be possible, just asynchronous (oops geekspeak).

For example, if I had had 1. permissions to create repositories, and 2. group consensus, I would have already made progress on docker with ned (see slack thread), and also the site.

thanks!
-r

@Robbt
Copy link
Member

Robbt commented Nov 7, 2018

I'm onboard with @ned-kelly becoming a maintainer. I am excited that we are growing the project. I'm not sure how to do all of the stuff behind the scenes as far as github goes, but it might make sense to have a maintainers coordination discussion. We could do it on here, via RT chat (might be tricky considering the timezone differences) or on the forum.

I think the trickiest thing at this point is figuring out who is going to do what. For instance in this particular PR should we redo it to include the commits in @hairmare added and if so what exactly are the steps to do this.

@frecuencialibre
Copy link
Contributor Author

I'm onboard with @ned-kelly becoming a maintainer.

consensus! could you go ahead and do this @Robbt ?

it might make sense to have a maintainers coordination discussion.

definitely. very necessary. weekdays in the mornings (CST, or UTC/GMT -6 hours) are generally best but i can be flexible. perhaps in slack? are audio conference calls possible in slack?

should we redo it to include the commits in @hairmare added

i personally don't think the site commit history is necessary. Ned's docker repo is however a different story -- with both a meaningful commit history and issues that would be good to keep.

take care everybody!

@Robbt
Copy link
Member

Robbt commented Nov 8, 2018

Great, I sent the invite to @ned-kelly

I personally think that the commits might be useful to show what changes were made from the default template but it probably isn't too important.

As far as transferring issues it appears that just last month this was added to Github beta so I think we can probably use it - see this discussion and here's the doc

@Robbt
Copy link
Member

Robbt commented Nov 8, 2018

I just create a PR with @hairmare work that retained the @ned-kelly commits at #590 - since that was easy to do. I'm not sure if it is ready to merge as it includes some work on the travis CI config. If we decide to merge that we can just close this one.

@frecuencialibre
Copy link
Contributor Author

I just create a PR

awesome! closing this one.

As far as transferring issues

my understanding of @ned-kelly 's suggestion is to simply transfer the entire repo. however he'd need repository creation permissions. would that be a problem? i think maintainers team members currently do not have that permission:
screenshot from 2018-11-09 10-43-25

thanks!
-r

@Robbt
Copy link
Member

Robbt commented Nov 11, 2018

It looks like in order to give @ned-kelly the ability to do this we would need to enable it for all members since he is not an owner and there doesn't appear to be a way to just give this permission to members of a certain team vs. all members at least from my limited research. We could do this temporarily so that it could be transferred and then turn off the permission after it is done. In general "membership" is pretty open vs. membership on a specific team. It is also possible that I just couldn't find the right way to say give maintainers only the ability to create repositories (which I figure would be ideal).

@frecuencialibre
Copy link
Contributor Author

frecuencialibre commented Nov 11, 2018 via email

@hairmare
Copy link
Member

https://github.com/ned-kelly/libretime-website doesn't seem to have any issues on it. Did I miss something that didn't make it into #590.

IMO #590 looks great and is waiting for @ned-kelly to return from his holiday so we can focus on reviewing and landing it.

I already rebuilt the travis parts and there is some potential to get rid of the included dependencies since the build process will redownload them anyway.

@lock
Copy link

lock bot commented Nov 11, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please chat to us on discourse or ask for help on our chat if you have any questions or need further assistance with this issue.

@lock lock bot locked as resolved and limited conversation to collaborators Nov 11, 2019
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.

None yet

4 participants