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

doc: adding example with compiling code for easier use for new users of skewer #22

Merged
merged 2 commits into from
Oct 6, 2023

Conversation

Bryce-Soghigian
Copy link
Contributor

** Why was this PR Needed?**
The introductory example did a good job of showing off some new methods, but requires the user to have some baseline understanding of what variables to set to get the project up and running, and the azure new credential from environment hides some of this behind some abscraction.

So i opted into putting the variables there, and having the main.go script runnable so the users can get a compiled example from the get go to start experimenting with the package

@codecov-commenter
Copy link

Codecov Report

Merging #22 (67b9c23) into main (991f172) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   72.32%   72.32%           
=======================================
  Files           6        6           
  Lines         336      336           
=======================================
  Hits          243      243           
  Misses         93       93           

README.md Outdated Show resolved Hide resolved
@Bryce-Soghigian Bryce-Soghigian merged commit 6f2365e into main Oct 6, 2023
2 checks passed
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