| Previous Lecture | Lecture 24 | Next Lecture |
Lecture 24, Wed 03/02
Wed Lecture: work on team04: plan to have some PRs ready by Monday (sooner if possible)
Today:
Standup on team04
- Always start with a standup meeting
- By now, you should all have an issue groomed, and be working on it
- Estimate when you’ll have a PR ready for the team
Review Yesterday’s Goals
Yesterday, you should have done all of the things in the list below.
- If you did, hooray, skip this part.
- If you didn’t, then get caught up.
The list from Yesterday:
- At a minimum, create the columns
Todo,In Progress,In Review,Doneon your team’s board. - We suggest that you consider splitting the
In Reviewcolumn into two columns:Team Review: Stories/Issues/PRs that are ready for the team to reviewStaff Review: Stories/Issues/PRs that are ready for the staff to review. This should make it easier to determine where the PRs/Issues are in the process, and who needs to take action to move things along. (Thanks Kevin H. for this suggestion.)
- Create at least a few issues in your repo
- NOTE that since you SHARE a repo with another team, your “Issues” column is shared with that team.
- You can find sample issues under the
/issuesdirectory in your team’s repo, underepic01,epic02,epic03, orepic04 - Unlike in previous assignments, it is now your team’s responsibility to create the Issue in GitHub, and “groom” it (i.e. add Acceptance Criteria)
- For grooming of issues, TA Kevin Heffernan suggests this resource (about an 8 minute read): https://www.productplan.com/glossary/backlog-grooming/
- Make sure that each team member is assigned to at least one in-progress issue
That’s the minimum bar for today’s discussion section. If you have time to do more, some suggesitons appear below the table of Kanban boards. There are also a few reminders.
Same reminders as yesterday
- Unlike in previous team exercises, you do not have full admin control over the repo.
- In particular, you do not have access to push or merge directly into the default (
main) branch.- It is not universal, but pretty common in industry that devs don’t have direct access to push to
master/main - Instead, there is a “process” that you have to go through to get stuff merged into
master/main
- It is not universal, but pretty common in industry that devs don’t have direct access to push to
- The process in this project is also the way you earn points.
- Each team shares a grade for the project, out of 100.
- You may earn up to 110 points; extra 10 count as extra credit (2.5% of course grade, max).
- What you need before a PR can be merged:
- Linked to an issue
- Issue is groomed with Acceptance Criteria
- All Acceptance criteria are met
- PR has good description
- All CI/CD checks pass, including mutation testing
- Approving code review from member of team that didn’t work on the issue
- Approving code review from a staff member
New things to do
Here are some things to do if you have time to go beyond the minimum
- Get your qa deployments are set up. See Slack discussion in either
#help-proj-courses-searchor #help-proj-happycows - Make sure you can clone the repo and get it up and running on localhost. The discussion at the previous link has some hints about reusuing your OAuth credentials.
- Talk with the staff about the “big picture” of your sprint.
- That is, try to understand what it is you are trying to build.
- What is in scope, and what is out of scope?
- What does the end user want/need from this feature?
- For courses search, Prof. Conrad, Andrew, Pranav are the ones most familiar with the app
- For Happy Cows, Seth, Kevin, Bryan Z are the best sources
- Bryan T is equally familiar with both.
The rest of this is repeat from Yesterday
But if you didn’t read it yesterday, read it now.
You may need to rebase on main often
You are now working in a repo where more than just your own team is working.
This increases the chance for merge conflicts.
This means that you should rebase on main early and often.
The complexity of rebasing on main can range dramatically:
- In some cases, super easy!
- In others, mind-bogglingly hard
It isn’t possible to write a full guide for every possible permutation of what can happen. But we’ll provide a guide to the simple cases, and then encourage you to reach out for help if and when the complex ones arise.
Here’s the article: https://ucsb-cs156.github.io/topics/git_pull_rebase_main/
Some high level observations
Courses Search and Happy Cows are in vary different stages of development
For, Courses Search, this is our third major version of the app: some of the staff have been working on it for over two years now, and we really know the features very well, inside and out.
For Happy Cows, we are at a much earlier stage. Some of the dev team (Conrad, Bryan T, and Seth V) did some maintenance programming on an earlier version of the app, and then started on a re-write using this tech stack in S21. But we didn’t very far, and there are still many things that are still being defined.
Thie means you’ll have very different experiences, with different tradeoffs.
Courses Search will be a more straightforward path; the user/customer/product manager has a much clearer idea of what they want, and the staff has a pretty clear idea how to build that.
Happy Cows will be less straightforward. There is less clarity about what the finished product should look like, or the best way to get there.
You may think that this means that Courses Search folks are the more fortunate ones. I’d suggest, though, that the experience of the Happy Cows folks is much more authentic and typical of real world coding. They may have the better learning experience.