-
Notifications
You must be signed in to change notification settings - Fork 66
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
high level restructuring #504
Conversation
@pachadotdev, I'm promise I won't add anymore to this PR {edit: tonight}. I just kept seeing more stuff. Please tell me if you like the changes, and feel free to overrule me. This will close #501 |
@pachadotdev, I've incorporated @zeileis's advice, which will close #500. See if you like it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is great
@wibeasley I merged with a rebase (i.e., to keep a clear log) |
@pachadotdev, it's not a big deal, but I wanted you to know that some contents of this file were lost --maybe during the rebase? (PR #504 (comment)) It was pretty easy to recover from this commit though (from right before the PR) https://github.com/cran-task-views/WebTechnologies/tree/fbdfc3fc51ca9806ae50dad993a2a5904ae1fb8a
closes #502