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

init: rewrite init process #260

Merged
merged 5 commits into from
Jun 1, 2016
Merged

init: rewrite init process #260

merged 5 commits into from
Jun 1, 2016

Conversation

derekstavis
Copy link
Member

Now use pure globbing to generate 100% valid function and
completion paths, effectively splitting the init process in two
steps, one which paths are added, and other when initialization
is done (sourcing init).

This initialization code introduces a new interface for
init.fish hook, which deprecates the previously used event
model. The new interface injects three variables into init.fish:
path, package and bundle. This variables can be used by the
package to autoload paths, use bundled files, etc.

Also supports key bindings by sourcing
$OMF_CONFIG/key_bindings.fish and also key_bindings.fish in
packages (plugins and themes) root directories. This is done
when fish_user_key_bindings is called.

@sagebind
Copy link
Member

sagebind commented Mar 4, 2016

Wow, just the code looks so much better and more efficient! Great job with this, @derekstavis!

@sagebind
Copy link
Member

sagebind commented Mar 6, 2016

I have no issues when running this change. Everything appears to be working excellently without errors, and it starts up drastically faster. Like, it's amazing. Great work on all of this!

@sagebind
Copy link
Member

What is the progress on this? Are we reaching merging time?

@derekstavis
Copy link
Member Author

I have updated this branch to reflect the changes introduced by leaner-init branch, which undergone some structural changes, as the removal of initialize function.

@sagebind sagebind added the priority: high High priority label Mar 31, 2016
@derekstavis derekstavis force-pushed the better-init branch 2 times, most recently from fa93faf to 2402175 Compare March 31, 2016 02:00
@derekstavis
Copy link
Member Author

@oh-my-fish/docs please review docs - It was done using Google Translate

@demidovakatya
Copy link
Contributor

@derekstavis

pleaser review docs - It was done using Google Translate

what do you mean?

@derekstavis
Copy link
Member Author

Sorry for my brevity, I was on mobile. This PR touches translated docs. I've done an automatic translation from English to other languages using Google Translate. I'm pinging @oh-my-fish/docs team to review the changes contained on this PR, so it can be merged with correct docs.

@demidovakatya
Copy link
Contributor

@derekstavis, thanks for your explanation! I can translate the changes to Russian, is it needed?

Now use pure globbing to generate 100% valid function and
completion paths, effectively splitting the init process in two
steps, one which paths are added, and other when initialization
is done (sourcing init).

This initialization code introduces a new interface for
`init.fish` hook, which deprecates the previously used event
model. The new interface injects three variables into `init.fish`:
path, package and bundle. This variables can be used by the
package to autoload paths, use bundled files, etc.

Also supports key bindings by sourcing
$OMF_CONFIG/key_bindings.fish and also key_bindings.fish in
packages (plugins and themes) root directories. This is done
when fish_user_key_bindings is called.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants