Skip to content

Conversation

@da2ce7
Copy link
Contributor

@da2ce7 da2ce7 commented Nov 29, 2022

Format all files in the Index Backend.

@josecelano
Copy link
Member

hi @da2ce7, I think we are again in the same situation :-). PR #77 is done, and It probably has a lot of conflicts with this one. In this case, it contains a lot of tests, but only for the new feature (upgrader).

But I think PR #77 contains a lot of additions and few modifications. I suppose we can merge any of them first. Let me know which one you prefer merging first. Besides, I would like to merge both of them before starting to work on the next issue.

@da2ce7
Copy link
Contributor Author

da2ce7 commented Nov 29, 2022

@josecelano Unlike the other case. I would like to merge this one first. The formatting changes made your code hard to review.

@da2ce7 da2ce7 force-pushed the format-world branch 7 times, most recently from 58f075e to a39a0d9 Compare November 29, 2022 14:23
@da2ce7
Copy link
Contributor Author

da2ce7 commented Nov 29, 2022

ACK a39a0d9

@da2ce7 da2ce7 merged commit a5fcdeb into torrust:develop Nov 29, 2022
@josecelano josecelano mentioned this pull request Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants