Skip to content
This repository has been archived by the owner on Nov 25, 2022. It is now read-only.

Add i18n diff check for Element desktop #142

Merged
merged 1 commit into from
Apr 27, 2021
Merged

Conversation

germain-gg
Copy link
Contributor

@germain-gg germain-gg commented Apr 26, 2021

@dbkr
Copy link
Member

dbkr commented Apr 27, 2021

Presumably this doesn't want to be merged until the element PR as are merged

@germain-gg
Copy link
Contributor Author

Presumably this doesn't want to be merged until the element PR as are merged

Very true, I have updated the PR comment to add a little "blocked by" mention!

Copy link
Contributor

@jryans jryans left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me, thanks!

@germain-gg
Copy link
Contributor Author

@dbkr I do not have sufficient permissions to merge this.
element-hq/element-desktop#188 has landed, could you merge this PR?

@kegsay kegsay merged commit e617067 into master Apr 27, 2021
@kegsay kegsay deleted the gsouquet-i18n-electron branch April 27, 2021 15:34
@kegsay
Copy link
Member

kegsay commented Apr 27, 2021

@gsouquet done

jryans added a commit that referenced this pull request May 5, 2021
#142 added an extra CI step, but
since the plugins are part of each step, that change meant that the pre-existing
lint step no longer had a Docker image.

This fixes things up by setting up the plugins for each step.
richvdh pushed a commit that referenced this pull request May 5, 2021
#142 added an extra CI step, but
since the plugins are part of each step, that change meant that the pre-existing
lint step no longer had a Docker image.

This fixes things up by setting up the plugins for each step.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants