search

nus-cs2103-AY2122S2 / forum

13 stars 1 forks source link

Error when merge pr (Java CI problem) #211

Closed yumengtan closed 2 years ago

yumengtan commented 2 years ago

I am facing this problem when I merged the pr. The pr passed the CI test. May I know what caused this error and how to fix it? image More details here

Punpun1643 commented 2 years ago

I had the exact same error before when everything passed the CI test. Turned out it was some styling issue/ spacing issue in my fxml file and css file. Maybe you could look into that as well 😄

damithc commented 2 years ago

@yumengtan still not resolved? Are you the only one in your team facing this error? Is this the PR in concern? https://github.com/AY2122S2-CS2103-F11-4/tp/pull/38 Seems it is merged already.

damithc commented 2 years ago

@yumengtan Good to hear the issue is resolved. You can go ahead and close the issue now.

yumengtan commented 2 years ago

@damithc After trying to re-run the merge pr, it seems to be working fine now.