Skip to content

Conversation

@mickvandijke
Copy link
Member

Rebased and processed @josecelano 's feedback: #100

@mickvandijke mickvandijke requested review from da2ce7 and josecelano May 3, 2023 14:28
Copy link
Member

@josecelano josecelano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @WarmBeer, it looks good to me.

Cargo.toml Outdated
Comment on lines 44 to 46
bytes = "1.4.0"
text-to-png = "0.2.0"
indexmap = "1.9.3"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @WarmBeer, I changed all dependencies to use only the minor versions so that you can upgrade patches.

bytes = "1.4"
text-to-png = "0.2"
indexmap = "1.9"

@josecelano josecelano added the Needs Rebase Base Branch has Incompatibilities label May 3, 2023
@mickvandijke mickvandijke force-pushed the rebase-91-add-endpoint-for-image-proxying branch 2 times, most recently from 39a6218 to dc279db Compare May 4, 2023 10:44
@mickvandijke mickvandijke force-pushed the rebase-91-add-endpoint-for-image-proxying branch from dc279db to 916d869 Compare May 4, 2023 11:04
@mickvandijke
Copy link
Member Author

Hi @josecelano ,

I still have 1 failing test after rebasing. Is there something else I need to do?

@josecelano
Copy link
Member

Hi @josecelano ,

I still have 1 failing test after rebasing. Is there something else I need to do?

@WarmBeer, that's the test I knew would fail because you added a new section to the configuration. Let's merge the PR, and I will fix it since I will start working on this issue, which is related to configuration. But just for let you know, you only need to add the new section in here.

@josecelano
Copy link
Member

Hi @josecelano ,
I still have 1 failing test after rebasing. Is there something else I need to do?

@WarmBeer, that's the test I knew would fail because you added a new section to the configuration. Let's merge the PR, and I will fix it since I will start working on this issue, which is related to configuration. But just for let you know, you only need to add the new section in here.

I'm fixing it in a new branch with a new commit. I will replace this PR with the new one in a few minutes.

@mickvandijke mickvandijke merged commit 7cd0cb2 into torrust:develop May 4, 2023
@mickvandijke mickvandijke removed the Needs Rebase Base Branch has Incompatibilities label May 4, 2023
@josecelano
Copy link
Member

@WarmBeer this is the new PR fixing the test.

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