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

Change set_dir and login functions to read DB credientals from enivorment variables. #28

Open
novica opened this issue Sep 11, 2023 · 4 comments

Comments

@novica
Copy link
Member

novica commented Sep 11, 2023

There is going to be a change in the login to PJS this weekend or next. That may be a nice time to change this. As far as I can see we only need to change a few things
Get the information into .Renvirion, that should be mostly done so that people don't have to type themselves.
Change NVIdb::set_dir_NVI
Change NVIdb::login (group of functions, but they are partly wrappers around one or two core functions. Probably one should create a "subfunction" to retrieve the data in .Renviron.

@novica
Copy link
Member Author

novica commented Sep 11, 2023

The way we are going to do this is all changes are going to be made via a pull requests on the dev branch of the repository. From the dev branch we create additional branches, usually named after the issue (in this case I will create a branch called issue-28). I will implement the changes on that branch, then I will make a PR on dev. Once that is verified, the package maintener can merge dev into main. Right now there are no automatic checks set up, but this is something that can be done to make sure the new code doesn't crash something.

@PetterHopp
Copy link
Member

Sorry Novica, I accidentally deleted the branch. I thought it was something else and didn't check. How much did you loose and do you have a copy of it?

@novica
Copy link
Member Author

novica commented Jan 24, 2024 via email

@PetterHopp
Copy link
Member

Thanks. I will make a new directory with nearly the same name.

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

No branches or pull requests

2 participants