Closed fepegar closed 3 years ago
Feel free to merge by yourself? Or we want to do code review now already?
Given the collaborative spirit of the project, I think it'd be nice to have at least one review from someone else. Also, I'd always use "Squash and merge" (relevant when there are multiple commits in the PR). What do you think?
Given the collaborative spirit of the project, I think it'd be nice to have at least one review from someone else. Also, I'd always use "Squash and merge" (relevant when there are multiple commits in the PR). What do you think?
Considering the nature of this repo, yes, squash should be good, will set it in the repo later.
Usage:
It takes an optional modality: T1, T2, PD, MRA... The default is T1.