-
-
Notifications
You must be signed in to change notification settings - Fork 324
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
Refactor Feature/Module/connector => "Plugin" (Third try) #593
Conversation
First step of Plugin refactor. Preserving file history, however, tests will fail for this commit.
Including files in /src/client, src/server and tools
Improves clarity and eases the next step of the plugin refactor
Renames all references to the Feature class
* Rename variables and class names to plugin[s], and Plugin * Correct references to files renamed in previous steps
@aneilbaboo It cannot be merged, because it is not only couple graphql files, I see many files in the tools dir where Git has lost track of renamings |
In your last commit: 66d236f |
@aneilbaboo I'm looking to have two commits in this pull request. The first commit does all the renamings. The second commit changes all the files contents as needed. Please be careful with that and use |
@Vlasenko - I see what's been happening now. I created a branch, on my repo, then merged that branch into my master on Github. Github doesn't do a fast forward merge. It squashes all the commits together. Let me fix it. |
@aneilbaboo Lets hold off with that. I think I should document current approach carefully first, then we will think about better naming. I plan to do so on the second week of January, as I'm on a vacation right now |
@aneilbaboo Will |
Maybe plugins is a good name. Imagine you have npm-like repo and you install these plugins into your project. To install them you need to have right |
@Vlasenko - I'll hold off on anymore changes. I think |
@aneilbaboo Yes, I think |
81fe3d5
to
a2b8b27
Compare
22cc234
to
06fd4b4
Compare
See discussion at #565
Renamed modules folders to plugins
Renamed connector.js to plugin.js
Renamed Feature class to Plugin
Renamed modules variables to plugins
Renamed misc other Module___.___ files to Plugin___.___
@Vlasenko - This is the third try. I was careful to do directory renamings first, followed by file renamings, then alterations of file contents, each as separate commits, but git/Github is treating a couple of graphql files as different. It's weird because neither the filename nor contents changed, just the directory. I've spent a few hours on this taking different approaches, but it's not working.
Can we get this merged?