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

Include Github https repository getter utility #243

Open
wants to merge 1 commit into
base: master
from

Conversation

@alwell-kevin
Copy link
Contributor

alwell-kevin commented May 28, 2019

Includes a javascript utility which enables a developer to GET a private GitHub Repository from an Organization over HTTPS (no GIT) using the CLI.

@azizshamim

@azizshamim azizshamim self-requested a review May 28, 2019
@jonico
jonico approved these changes Mar 20, 2020
Copy link
Contributor

jonico left a comment

See minor change suggestions inside

const spiner = new Spinner('Retrieving your Project, please wait...');
spiner.start();

github.getRepo(gitHubMeta, spiner).then(function(repo) {
console.log("Got Project: ", gitHubMeta.repo)
spiner.stop();
Comment on lines +25 to +30

This comment has been minimized.

@jonico

jonico Mar 20, 2020 Contributor

Suggested change
const spiner = new Spinner('Retrieving your Project, please wait...');
spiner.start();
github.getRepo(gitHubMeta, spiner).then(function(repo) {
console.log("Got Project: ", gitHubMeta.repo)
spiner.stop();
const spinner = new Spinner('Retrieving your Project, please wait...');
spinner.start();
github.getRepo(gitHubMeta, spinner).then(function(repo) {
console.log("Got Project: ", gitHubMeta.repo)
spinner.stop();
var download = require("download");
var request = require('request');

var getRepo = (inputs, spiner) => {

This comment has been minimized.

@jonico

jonico Mar 20, 2020 Contributor

Suggested change
var getRepo = (inputs, spiner) => {
var getRepo = (inputs, spinner) => {
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.