Skip to content

add definition for glob #931

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

Closed
wants to merge 1 commit into from
Closed

add definition for glob #931

wants to merge 1 commit into from

Conversation

dongzhuoer
Copy link
Contributor

maybe the order can be adjusted as

// Obtain an R environment
Environment stats("package:stats");
Environment env( 2 ); // by position
Environment glob = Environment::global_env();

// Special environments
Environment::Rcpp_namespace();
Environment::base_env();
Environment::base_namespace();
Environment::empty_env();

@eddelbuettel
Copy link
Member

Please STOP submitting PRs now and read this document about process first

One small one I let slip, two I am not pleased with. Three is clearly out of bounds.

All these are in the same effing document. Please pool resources and discuss first as the Contributing.md document you are being shown on each PR you make asks for.

@dongzhuoer
Copy link
Contributor Author

I'm so sorry.

@dongzhuoer
Copy link
Contributor Author

dongzhuoer commented Dec 22, 2018

Moved to #933

@dongzhuoer
Copy link
Contributor Author

dongzhuoer commented Dec 26, 2018

implemented in 6100a55

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.

2 participants