Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spelling: ": c" → ": C" and "tag" → "Tag" #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

comradekingu
Copy link

No description provided.

@info-cppsp
Copy link

debug messages generally don't matter much.
If they have a spelling / grammatical error, than sure, but uppercase, lowercase?
The tag is a translated string, that means (?) that it would have to be translated again.
So if I had to decide, I would reject this PR.
Guys?
@monsta @raveit65
This is a small change, so it should be dealt with soon, right?

@raveit65
Copy link

Can, we wait a bit?
I'd like to change transfex configs from master to latest stable.
I will start a discussion about next week, because i am not 100 % fit again after the flu.

@raveit65
Copy link

The problem is that origin author never contact us or react on invitation after we cloned his repo last year.
https://github.com/fbarriga/caja-xattrs
see also this PR mate-desktop/caja-extensions#33

The deal was that he active maintain this package as maintainer!
But he didn't.
For this reason we didn't release or tag his package for 1.20
For me and others this package isn't important enough and i don't want to maintain it.
We have enough other packages and the team is very small.

@info-cppsp
Copy link

should I write it up, that this repo should be considered to be dropped?

well, if it is like you say, I would suggest rejecting this PR.

@raveit65
Copy link

I was the one who powered to include this third party repo + recruit a new developer.
But for me this is a deal breakage in result i stopped my support.
We should discuss this as team discussion, maybe someone have the time and like to maintain it.
If we drop it we can close the PR, and remove it from transifex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants