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

Add Hacktoberfest 2020 guidance from cdf #196

Merged
merged 1 commit into from
Oct 15, 2020

Conversation

a-roberts
Copy link
Member

@a-roberts a-roberts commented Sep 4, 2020

This has a few Dashboard ideas I can make issues for if they don't exist, and some guidelines so if anyone does want to pitch in to Tekton and become a contributor, they have an idea of where to start.

I'll work on fleshing out the Dashboard ideas too, hopefully with @AlanGreene and @eddycharly!

Would love for more folks to pitch in with cool ideas for each project area next 😄

This is for #188

Update: I agree with @eddycharly's proposal, decentralised this, provided guidance, and a link to the CDF's page for goodies

@tekton-robot tekton-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Sep 4, 2020
@tekton-robot tekton-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 4, 2020
@a-roberts a-roberts force-pushed the hacktoberfest branch 2 times, most recently from b0cdf63 to 767bb69 Compare September 4, 2020 15:43
@AlanGreene
Copy link
Member

For the Dashboard issues:

Added an item to the Dashboard WG agenda for next week too.

@a-roberts a-roberts force-pushed the hacktoberfest branch 2 times, most recently from 99390c0 to 331988c Compare September 7, 2020 08:56
@a-roberts
Copy link
Member Author

For the Dashboard issues:

Added an item to the Dashboard WG agenda for next week too.

Updated, thanks

@eddycharly
Copy link
Member

eddycharly commented Sep 7, 2020

Regarding the Dashboard, one contribution area could be with translations.

Adding a sort of theming support (light and dark themes for example) could be fun too.

Ah, and there's pipeline run graph support too.

@a-roberts
Copy link
Member Author

a-roberts commented Sep 8, 2020

Regarding the Dashboard, one contribution area could be with translations.

Adding a sort of theming support (light and dark themes for example) could be fun too.

Ah, and there's pipeline run graph support too.

Good points, I'll add entries for translation support (I know @AlanGreene has a PR for this tektoncd/dashboard#1712), themes, and graphs now 😄 we can eventually then get some issues made and linked here

I'm wondering if anyone from the other project ideas wants to pitch in with their thoughts for something new, let's poke

@dibyom @vdemeester @afrittoli @abayer @bobcatfish

ideally it won't just be "awesome stuff we want people to work on" but "awesome stuff that may be interesting for first timers and is a little bit out there/different"

@eddycharly
Copy link
Member

Not sure if having tables here listing issues is going to be practical to manage.

Maybe we should have a simple welcome guide here, and add an hacktoberfest label to the relevant issues in each repo ?

In the welcome guide we can present each project briefly and explain newcomers how to find the relevant issues, how to assign an issue to themself so that someone else knows that it was already picked up, etc...

Then in the issue itself, we could state things like the level of difficulty of the issue (not all issues are going to be easy to implement), some background infos (previous work, challenging areas), some contact information, relevant docs, etc...

I mean, the more the issue is self contained the better, no ?

@a-roberts
Copy link
Member Author

a-roberts commented Sep 8, 2020

Not sure if having tables here listing issues is going to be practical to manage.

Maybe we should have a simple welcome guide here, and add an hacktoberfest label to the relevant issues in each repo ?

In the welcome guide we can present each project briefly and explain newcomers how to find the relevant issues, how to assign an issue to themself so that someone else knows that it was already picked up, etc...

Then in the issue itself, we could state things like the level of difficulty of the issue (not all issues are going to be easy to implement), some background infos (previous work, challenging areas), some contact information, relevant docs, etc...

I mean, the more the issue is self contained the better, no ?

Yeah, I agree, updating this is proving irritating as it is and a decentralised approach is much better. I'll change this to contain guidance for both soon-to-be contributors and maintainers, and get the labels made for the projects I can (Dashboard and Experimental I think)

@a-roberts a-roberts force-pushed the hacktoberfest branch 2 times, most recently from 22dc607 to 8902723 Compare September 8, 2020 11:15
@a-roberts a-roberts changed the title [WIP] Add starting point for Hacktoberfest project ideas [WIP] Add starting point for Hacktoberfest guidance Sep 8, 2020
@a-roberts
Copy link
Member Author

@eddycharly done, and the notes I had for Dashboard ideas are (here for brevity so we can go about making issues and labelling them):

Docs / website / katacoda enhancements
Import
from catalog by version?
Kubectl apply doesn’t work with generateName (needs to be a create) which is used in a lot of examples in the pipeline repo, so should the action be customisable or use a different method to import the resources? https://github.com/tektoncd/dashboard/issues/1552
Support git revision https://github.com/tektoncd/dashboard/issues/1550
Workspaces not supported for TaskRuns and PIpelineRuns https://github.com/tektoncd/dashboard/issues/1283
Anything for chains? Looks like no CRDs
Single namespace install revisit (docs) https://github.com/tektoncd/dashboard/issues/1018 and https://github.com/tektoncd/dashboard/issues/1507
Security revisit (anything preventing adoption)
Platform support - try and doc
Any links/common themes to the other Tekton project areas?
Update roadmap with progress to date / changes in approach etc.

@eddycharly
Copy link
Member

Thanks, i created issues for theming and pipeline run diagrams.

@a-roberts
Copy link
Member Author

I think we're all done here except for the todos, I'm hoping to follow up on those with the CDF, but would like other projects to start thinking of doing the same thing with the labels next 🐱

@a-roberts a-roberts changed the title [WIP] Add starting point for Hacktoberfest guidance Add Hacktoberfest 2020 guidance from cdf Oct 2, 2020
@tekton-robot tekton-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 2, 2020
@a-roberts a-roberts force-pushed the hacktoberfest branch 2 times, most recently from 693e51d to 87717b4 Compare October 2, 2020 15:22
@AlanGreene
Copy link
Member

Here's hoping for quality contributions :octocat: #hacktoberfest
/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 2, 2020
@popcor255
Copy link
Member

There are some changes we need to make if we want tekton to be opt in hacktoberfest. https://hacktoberfest.digitalocean.com/hacktoberfest-update

@a-roberts
Copy link
Member Author

There are some changes we need to make if we want tekton to be opt in hacktoberfest. https://hacktoberfest.digitalocean.com/hacktoberfest-update

Good catch, I don't have the option to add topics for repos, perhaps something we can do at a higher level? @bobcatfish or @vdemeester perhaps?

@tekton-robot tekton-robot removed the lgtm Indicates that a PR is ready to be merged. label Oct 7, 2020
@vdemeester
Copy link
Member

Good catch, I don't have the option to add topics for repos, perhaps something we can do at a higher level? @bobcatfish or @vdemeester perhaps?

We can indeed add the topic on projects 👼 and labels 😉
@a-roberts can you open a PR updating https://github.com/tektoncd/plumbing/blob/master/label_sync/labels.yaml so that we create those labels ? once this is done, let's get this one merged and add the topics 😉

@a-roberts
Copy link
Member Author

Good catch, I don't have the option to add topics for repos, perhaps something we can do at a higher level? @bobcatfish or @vdemeester perhaps?

We can indeed add the topic on projects 👼 and labels 😉
@a-roberts can you open a PR updating https://github.com/tektoncd/plumbing/blob/master/label_sync/labels.yaml so that we create those labels ? once this is done, let's get this one merged and add the topics 😉

Thanks for taking a look, how about tektoncd/plumbing#607? I haven't tested it though 🤞 getting late here so will take another look tomorrow if there are problems

@a-roberts
Copy link
Member Author

/meow

@afrittoli @vdemeester please 🙏

@tekton-robot
Copy link
Contributor

@a-roberts: cat image

In response to this:

/meow

@afrittoli @vdemeester please 🙏

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@vdemeester
Copy link
Member

/approve

@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 9, 2020
@vdemeester
Copy link
Member

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 15, 2020
@tekton-robot tekton-robot merged commit a061ebf into tektoncd:master Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants