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

Initial changes from Daniel Jennings #11

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

gorlak
Copy link
Owner

@gorlak gorlak commented Dec 19, 2019

No description provided.

danieljennings and others added 12 commits December 19, 2019 10:05
Remove build status
"archive" depots. I'm just treating those as remote depots, as you can't
even list the files in such a depot without a different command, e.g.
'p4 files -A //archive/...'. But it shows up in the depot list, at
least.
code like this that I don't have to, but my will is weak.
looked like I could just pass the actual Menu ID to LoadStringResource
and it would work, but all of the other spots found the correct string
directly, so I've done that.
as such in the delta tree. I've mostly found that I need to actually set
the change as the lparam for the change tree item if we have one,
otherwise we really don't have any context anywhere about anything.
to a couple of menu options (so they hide) but we don't use any of the
jobs/fixes stuff so I'd like to get it out of my sight and figured this
was the most appropriate way to do this.
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.

2 participants