Skip to content

feat: Set cf values to request headers for local debugging#35

Merged
gja merged 3 commits intogja:masterfrom
rita-liu:master
Jul 26, 2019
Merged

feat: Set cf values to request headers for local debugging#35
gja merged 3 commits intogja:masterfrom
rita-liu:master

Conversation

@rita-liu
Copy link
Copy Markdown
Contributor

Fix #34

@gja
Copy link
Copy Markdown
Owner

gja commented Jul 11, 2019

Hi, thanks for the PR.

I notice a that most of the tests are failing. Could you please update the PR with the tests fixed?

Here is an example of the failing test: https://dev.azure.com/tejas0372/tejas/_build/results?buildId=68

@gja gja changed the title Set cf values to request headers for local debugging feat: Set cf values to request headers for local debugging Jul 11, 2019
@gja
Copy link
Copy Markdown
Owner

gja commented Jul 19, 2019

Hi. Sorry for the late response. I've been a bit sick.

Could you do the following

  1. Add some tests to this PR for this new functionality (any one test is fine).
  2. Add your name to the contributors section of Readme.md?

I'll merge in the PR post that.

Thanks for the contribution! Excited to have this merged soon.

@rita-liu
Copy link
Copy Markdown
Contributor Author

Hi @gja, sorry for the bit of delay from me!

Updated the PR to include a new test and also readme. Hope it makes the functionality more clear. Let me know if any other changes are needed.

@gja gja merged commit 8d78e0a into gja:master Jul 26, 2019
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.

Set cf config values in request headers

2 participants