Skip to content

Update declaration #218

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

Merged
merged 8 commits into from
Jul 1, 2016
Merged

Update declaration #218

merged 8 commits into from
Jul 1, 2016

Conversation

ktsn
Copy link
Member

@ktsn ktsn commented Jun 19, 2016

Related to #215

I moved all type-related files to types directory.
And I added test cases for the d.ts file. You can run the test cases by executing the following command. (It's ok if there are no error is printed)

$ cd types
$ typings install # install d.ts of Vue.js
$ tsc -p test/tsconfig.json 

@yyx990803
Copy link
Member

Looks great - in the meanwhile, I'm trying to merge next branch with master for Vuex 1.0 which will support both Vue 1.x and 2.0. Would you mind rebasing after that?

@ktsn
Copy link
Member Author

ktsn commented Jun 24, 2016

Sure! I will rebase that.

dispatch(mutationName: string, ...args: any[]): void;
dispatch<P>(mutation: MutationObject<P>): void;

watch(path: string, cb: (value: any) => void, options?: WatchOption): void;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've merged next into master and this is the only thing that is deprecated. LGTM after removing this.

@ktsn
Copy link
Member Author

ktsn commented Jul 1, 2016

I've removed it :)

@yyx990803
Copy link
Member

Thanks! Sorry for the back and forth, but I just pushed some additional changes for 1.0 today. In brief:

Also see (the latest api reference)[https://github.com/vuejs/vuex/blob/master/docs/en/api.md].

@ktsn
Copy link
Member Author

ktsn commented Jul 1, 2016

OK, I'll add the types for them a little later!

ktsn added 3 commits July 1, 2016 12:32
- middlewares -> plugins
- new method: replaceState, on, once, off and emit
@ktsn
Copy link
Member Author

ktsn commented Jul 1, 2016

I'm done.

@yyx990803 yyx990803 merged commit acbb2dd into vuejs:master Jul 1, 2016
@yyx990803
Copy link
Member

Great work, thank you very much! Do we need to do anything to publish this to typings?

@ktsn
Copy link
Member Author

ktsn commented Jul 1, 2016

We need to send PR to typings/registry with a json file like this.
If you don't mind, I will do that too.

@ktsn ktsn deleted the types branch July 1, 2016 05:29
@yyx990803
Copy link
Member

@ktsn sure!

jaychsu added a commit to jaychsu/vuex that referenced this pull request Sep 7, 2016
* upstream/master: (52 commits)
  Update ja document (vuejs#240)
  small word correction (vuejs#257)
  Modify type files to external module definition format to publish them via npm (vuejs#242)
  update docs for 1.0-rc.2
  Fix typo of docs (vuejs#238)
  [release] 1.0.0-rc.2
  [build] 1.0.0-rc.2
  improve type test for subscribe
  update type definition for subscribe
  store.on(mutation) -> store.subscribe
  Update types for nested modules (vuejs#235)
  update plugins doc
  [release] 1.0.0-rc
  [build] 1.0.0-rc
  ignore explorations
  Update declaration (vuejs#218)
  add renamed logger
  include events methods in api reference
  expose replaceState()
  docs
  ...
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