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

Debugger: Default source context and default uniquifier #839

Merged
merged 7 commits into from
Jan 17, 2018
Merged

Debugger: Default source context and default uniquifier #839

merged 7 commits into from
Jan 17, 2018

Conversation

chingor13
Copy link
Contributor

@chingor13 chingor13 commented Jan 9, 2018

The extSourceContext field should be empty when there is no
source-contexts.json file available.

The default uniquifier should be based off a SHA1 of available files to
debug (.php files). We add the file size as a proxy for detecting
changes.

The extSourceContext field should be empty when there is no
source_contexts.json file available.

The default uniquifier should be based off a SHA1 of available files to
debug (.php files). We add the file size as a proxy for detecting
changes.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 9, 2018
@chingor13 chingor13 requested a review from tmatsuo January 16, 2018 23:46
\RecursiveRegexIterator::GET_MATCH
);

$files = array_keys(iterator_to_array($regex));

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@dwsupplee dwsupplee added the api: clouddebugger Issues related to the Cloud Debugger API. label Jan 17, 2018
@dwsupplee dwsupplee merged commit 12f641a into googleapis:master Jan 17, 2018
@chingor13 chingor13 deleted the default-source-context branch January 17, 2018 23:11
@dwsupplee dwsupplee mentioned this pull request Jan 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: clouddebugger Issues related to the Cloud Debugger API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants