lab-antwerp-1 / home

https://lab-antwerp-1.github.io/home/randomizer
MIT License
3 stars 13 forks source link

Alina: 03agile-development-week1-3 #82

Open AlinaTaoRao opened 3 years ago

AlinaTaoRao commented 3 years ago

my fork of the exercises repo: agile-group-3 materials agile-development

Study Plan (Suggested Study)

Curriculum Study Book


Learning Objectives

Week 1

Check-In

I Need Help With:

What went well?

What went less well?

Lessons Learned

Sunday Prep Work

AlinaTaoRao commented 3 years ago

Week 1

Check-In

I Need Help With:

What went well?

What went less well?

Lessons Learned

Sunday Prep Work

AlinaTaoRao commented 3 years ago

Week 2

Check-In

I Need Help With: npm spell-check.

  1. I clone the group repo
  2. create a branch alina-planning-constrains
  3. edit contrains.md
  4. commit edit
  5. run Code Quality Scripts , then I failed at npm spell-check. It looks like this picture below. I think it's a problem with html/css, but I didn't edit any html/css file. By the way, I passed the "lint-and-validate (pull_request)" check. Do I need to fix this problem? How? spell-check

What went well?

What went less well?

Lessons Learned

Trust your classmate and understand their situations.

Sunday Prep Work

arnochauveau commented 3 years ago

Hey @AlinaTaoRao concerning the npm spell-check issue: what version of node are you running (you can check by running node -v)? I had some issues as well while trying to run the command on my device. After using a newer Node version the problem was fixed. My errors looked like this:

TypeError: text.matchAll(...) is not a function or its return value is not iterable

If that was not the issue, could you please send the entire error log? You can just copy/paste the text.

gelilaa commented 3 years ago

@AlinaTaoRao please check this pull request for fixing your error

What went less well?

  • dev strategy-how to divide branches?

    • each page a branch? then how can 2 person collaborate to develop one page?

You are exactly right. one page is far too big to be one branch. The whole idea of Agile development is to deliver work in small, but consumable, increments.

  • or each issue a branch(that will be too much branches)?

I have made a simple example workflow to help you understand what yours should look like. Your branch should contain a small amount of work that your team can review and merge. the smaller the branch work the easier to review and avoid conflicts

  • design (Figma: from the very beginning)

You will be exploring Figma in the next module UX/UI as well. So no rush to understand everything but it is good to start now.

AlinaTaoRao commented 3 years ago

@arnochauveau Thanks for your advice. It worked. I upgraded my node.js form v14.17.6 to v16.13.0. The npm spell-check problem disappear. node upgrade to

AlinaTaoRao commented 3 years ago

@gelilaa Thanks for your feedback. I tried change the spell-check in package.json to "spell-check": "cspell \"./**\"", but it worked not well for me. I upgraded my node.js. There is no error any more . The example workflow is a very good. It solved my problem. 👍

AlinaTaoRao commented 3 years ago

Week 3

Check-In

I Need Help With:

What went well?

Lessons Learned

Read the clue in details and be patient. Try to fix lint problems.

Sunday Prep Work