Skip to content

Commit

Permalink
Merge pull request #5292 from sellout/delayed-int-roundtrip
Browse files Browse the repository at this point in the history
Add a transcript to show that #4711 has been fixed
  • Loading branch information
aryairani authored Aug 29, 2024
2 parents d3d3dbb + cc80583 commit b6e12d0
Show file tree
Hide file tree
Showing 2 changed files with 76 additions and 0 deletions.
19 changes: 19 additions & 0 deletions unison-src/transcripts/fix4711.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# Delayed Int literal doesn't round trip

```ucm:hide
scratch/main> builtins.merge
```

```unison
thisWorks = '(+1)
thisDoesNotWork = ['(+1)]
```

Since this is fixed, `thisDoesNotWork` now does work.

```ucm
scratch/main> add
scratch/main> edit thisWorks thisDoesNotWork
scratch/main> load
```
57 changes: 57 additions & 0 deletions unison-src/transcripts/fix4711.output.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,57 @@
# Delayed Int literal doesn't round trip

``` unison
thisWorks = '(+1)
thisDoesNotWork = ['(+1)]
```

``` ucm
Loading changes detected in scratch.u.
I found and typechecked these definitions in scratch.u. If you
do an `add` or `update`, here's how your codebase would
change:
⍟ These new definitions are ok to `add`:
thisDoesNotWork : ['{g} Int]
thisWorks : 'Int
```
Since this is fixed, `thisDoesNotWork` now does work.

``` ucm
scratch/main> add
⍟ I've added these definitions:
thisDoesNotWork : ['{g} Int]
thisWorks : 'Int
scratch/main> edit thisWorks thisDoesNotWork
☝️
I added 2 definitions to the top of scratch.u
You can edit them there, then run `update` to replace the
definitions currently in this namespace.
scratch/main> load
Loading changes detected in scratch.u.
I found and typechecked the definitions in scratch.u. This
file has been previously added to the codebase.
```
``` unison:added-by-ucm scratch.u
thisDoesNotWork : ['{g} Int]
thisDoesNotWork = [do +1]
thisWorks : 'Int
thisWorks = do +1
```

0 comments on commit b6e12d0

Please sign in to comment.