-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
🙌 Community Check-in #1: Summer of Code Week 2 #2739
Comments
Hello @stefannibrasil @milaaraujo @MargaretAN9 @Gauravano @namangupta01 @ViditChitkara @tech4GT @mridulnagpal @SidharthBansal @sagarpreet-chadha @Souravirus @MayankKashyap @siaw23 @ryzokuken @ccpandhare @icarito @abTest-123 @steviepubliclab @ebarry @publiclab/soc @publiclab/reviewers -- trying a new weekly check-in! 😄 👋 📣 |
Hi @jywarren @publiclab/soc 😁 |
Expecting to start soon! |
Hi @jywarren, @publiclab/soc, I am currently working on correcting the tests of the rails 5.0 upgrade |
@jywarren I would be a little blocked next week due to an important conference (Node's collab summit in Berlin), but would love to help out on the next week's check-in. 😄 |
Also, i know i've been asking everyone a lot for relatively tight formatting and structural changes -- being a little more picky now that we're tackling much bigger projects -- but i'm hoping the requests make sense and you see how more rigorous syntax and even stylistic changes can help make the code easier to review, test, and maintain! Thanks for your patience! |
i think @mridulnagpal had a question about how to test some new changes -- Mridul, want to link us to your question? |
@ryzokuken just added you to the list! You can copy some of this week's template and expand/modify/refine as you like! |
Hello Everyone! |
Hi, Naman - can you link us to that and we can review? Thanks!
…On Wed, May 23, 2018 at 1:23 PM, Naman Gupta ***@***.***> wrote:
Hello Everyone!
I completed the feature reply by email to comment, just waiting for the
reviews and getting merge and going to start working on the other features
also.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2739 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJ9yX2-ugW_xsPeyZQ0OgBQumKAgwks5t1ZsggaJpZM4UJX3p>
.
|
@NamanGupta can you paste the link here , so we can review... |
Here's the link! #2669 |
Awesome! I am so excited for the summer to start 🎉 I can help with the check -in's too. Are there any docs for this? |
Thanks, Steffani -- shall I add you further down so after you begin? And there's no docs or template right now because we just made this process up 😛 but you can start with this as a template, i guess -- open to ideas for what we should be asking each week! |
@MargaretAN9 @Gauravano @ViditChitkara @mridulnagpal @SidharthBansal @sagarpreet-chadha when you have a chance, can you check in with us? I'd like to hear from all SoC folks each week to stay in good touch! Thank you! |
hi @publiclab/soc 😃 , I am having my exams right now, but would try to push whole |
Hi @publiclab/soc team ! |
Can anyone please review #2737? |
Hi all! Just sharing a nice short overview of rails tests --
https://mixandgo.com/blog/feature-tests-vs-integration-tests-vs-unit-tests-in-ruby-and-rails
…On Thu, May 24, 2018, 8:40 AM Sidharth Bansal ***@***.***> wrote:
Can anyone please review #2737
<#2737>?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2739 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABfJ4dzudaVjpktxfcY8203CAibUvBzks5t1qoogaJpZM4UJX3p>
.
|
Thanks @sagarpreet-chadha -- added you to the list. @ryzokuken did you want to do this coming Monday, then, and @stefannibrasil the next or did you want to hold off until RGSoC begins? Either way! |
@jywarren I will also love to do the check-in. Can you explain a little what will be there in checkin? Any fixed time? I am currently working on the main logic of the authentication system. Facebook api has changed so it is giving me some trouble. Google api is working properly.Trying to correct fb api. I have made usertags as needed. Is there any other patch needed to be done with user tag? I am currently working on the user_sessions_controller. |
I'll wait, @jywarren :) |
OK, i wanted to say - we seem to have resolved the travis build issue now, so please go ahead and restart your builds. @Gauravano and @icarito can you chime in here with some notes on how to use And since @ryzokuken will be in a conference next week, so @sagarpreet-chadha did you want to do a Monday check-in just like this one? @SidharthBansal there is no template as this is a new idea -- but you could start with the sections I posted, and add anything you like. I'm sure it will evolve! I would like to ask folks two things -- First, if all @publiclab/soc fellows (not required for mentors, although you're welcome of course!) to check in each week, and Second, we have a pretty deep back-log of pull requests right now. If folks are stuck on something, or just have a bit of extra time, let's really try to get some of those completed and merged, so we have fewer to go through -- i think an ideal is fewer than 20! I know i've created a few... but let's everyone pitch in! If newcomers need help, let's support them!!! 😄 👍 Have a great weekend, folks! 🎉 |
Weekly check-in is an awesome idea @jywarren. Maybe we can have monthly call too..I will try to help as much I can with PR's and unresolved issues. Something about Unstable:
@icarito please add more to this list. Thanks. |
@jywarren @icarito please specify which commands to use while pushing on stable. As @namangupta01 and I, both are using above command and overrides each other's work. Thanks. |
Ok i will do the next check-in 👍 ! |
Ok, thank @gauravano |
Yes thanks everyone, I think we've had a great start! Please note we've recently made some changes to Makefile and have started to deploy using this file. So if your branch fails to build in unstable, you may need to rebase to latest master. Second, you can always check https://jenkins.laboratoriopublico.org/job/Plots-Unstable/ to see if it's been recently used and by whom. You'll notice somebody using it from the timestamps of previous builds. Do coordinate on #publiclab when using the unstable staging server. For pushing to unstable, from my working branch I usually to As @Gauravano pointed out, it will deploy to https://unstable.publiclab.org/. |
Closing out for #2750 --- thanks! |
Hello everybody! Welcome to Community Check-in number 1! If you're new here, welcome, and please checkout our welcome page!! 👍 🎉 😄
I'm just back from a big trip to California to help run a Community Science space at Maker Faire -- we helped over 1700 people make microscope, spectrometry, and water quality kits - https://twitter.com/PublicLab/status/999019548849434624
Check-ins
How are you all doing? What are you working on this week? Having trouble, have questions or stuck? Let's share!
CodeClimate
As we're getting started, you may see CodeClimate suggesting some changes -- please do your best to tidy up code as you go -- even if it's not yours! The checks look like this:
And it'll suggest areas where you could make some small improvements to the code. If you're waiting on a review or another response, these are a great place to just pitch in on the overall tidiness and maintainability of our project, OR to make a
first-timers-only
issue (read more here!) to help a newcomer take on one of these!Rotating community check-ins
I'd like to invite others to take turns doing the weekly community check-in -- preferably on Monday, although I was so busy I missed it! Say hey in the comments if you'd like to take next week's!
'Til next week!
As always, if you're waiting for a review, or if you're stuck, please request help
here OR leave a comment with @publiclab/soc and @publiclab/reviewers for
some input. We're stronger as a community!
Thanks everyone!
The text was updated successfully, but these errors were encountered: