Skip to content

Fix printing of type decls that use 'private' keyword#482

Closed
seliopou wants to merge 1 commit intoocaml:trunkfrom
inhabitedtype:trunk
Closed

Fix printing of type decls that use 'private' keyword#482
seliopou wants to merge 1 commit intoocaml:trunkfrom
inhabitedtype:trunk

Conversation

@seliopou
Copy link
Contributor

Specifically, this fixes the placement of the private keyword when a manifest is present.

type t = private A | B
type u = t = private A | B

Was previously printed out as:

type t = private A | B
type u = private t = A | B

  type t = private A | B
  type u = t = private A | B

Was previously printed out as:

  type t = private A | B
  type u = private t = A | B
@mshinwell mshinwell changed the title Fix printing of type delcs that use 'private' keyword Fix printing of type decls that use 'private' keyword Feb 24, 2016
@ghost
Copy link

ghost commented Feb 24, 2016

thanks, merged in trunk (7543d73) and 4.03 (7c2d0eb)

@ghost ghost closed this Feb 24, 2016
mshinwell pushed a commit to mshinwell/ocaml that referenced this pull request Jul 9, 2021
Coercions now carry recursion info as the "from" and "to" arguments
to `change_depth`. This means they carry names (in the form of
depth variables), which requires some care.
stedolan pushed a commit to stedolan/ocaml that referenced this pull request May 24, 2022
EmileTrotignon pushed a commit to EmileTrotignon/ocaml that referenced this pull request Jan 12, 2024
* Implement Card component

* Use Card component in Redirect

* Delete unnecessary unit arg

* Create a MediaObject.Small module

* Change type of imageSide to polymorphic variant

* Use MediaObject.Small in index

* Slightly simplify Tailwind module API

* Add some structure to storybook

* Add ShortWideCard

* Add HightlightsInQuadrants

* Add CallToAction.Embedded

* Fix build error

* Add StackedList.BasicWithIcon

* Add StackedList.BasicWithAuxiliaryAttribute

* Add CardGrid

* Add more examples

* expose Lang.ofStringExn

* Move wrapper div into MediaObject

* Add some more documentation
EmileTrotignon pushed a commit to EmileTrotignon/ocaml that referenced this pull request Jan 12, 2024
typo: availble -> available
This pull request was closed.
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.

1 participant