Skip to content

Conversation

@GrahamDennis
Copy link
Contributor

No description provided.

@iamdanfox
Copy link

Seems like wikipedia just redirects!

image

@carterkozak
Copy link
Contributor

Nice catch!

@iamdanfox
Copy link

I'm gonna try and connect this repo to the rest of our infra shortly (to get changelog-app etc working).

@iamdanfox iamdanfox merged commit 35533b4 into palantir:develop Nov 25, 2019
@iamdanfox
Copy link

Separately, I think you should have write permissions to this repo (i.e. can just make a branch) rather than requiring a fork!

@GrahamDennis
Copy link
Contributor Author

@iamdanfox No such luck:

08:43 $ git push
ERROR: Permission to palantir/human-readable-types.git denied to GrahamDennis.
fatal: Could not read from remote repository.

@iamdanfox
Copy link

iamdanfox commented Nov 25, 2019

Interesting. This repo is configured to give write access to everyone in https://github.com/orgs/palantir/teams/palantir-users/members, but I confirmed you're not in there. Looking internally, I think https://opensource.p.b/ is supposed to be the source of truth (and you are on that list). @bluekeyes are we doing something wrong here? Is Palantir Users not syncing correctly?

@bluekeyes
Copy link

Team membership in this org is not managed automatically by any systems at this time. Can you raise an internal ticket if that's something we should be doing?

@iamdanfox
Copy link

ACK - looks like the internal page shows ~553 users and the actual oss palantir-users team shows 229 members... was that a manually curated list previously?

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.

4 participants