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

Pre-requisite for a new release #88

Merged
merged 1 commit into from
Nov 29, 2018
Merged

Pre-requisite for a new release #88

merged 1 commit into from
Nov 29, 2018

Conversation

anuragsoni
Copy link
Collaborator

  • Switch to dune and opam 2 file formats
  • Try to remove usage of Lwt_log

Reference: #86
Signed-off-by: Anurag Soni anurag@sonianurag.com

* Switch to dune and opam 2 file formats
* Try to remove usage of Lwt_log

Reference: #86
Signed-off-by: Anurag Soni <anurag@sonianurag.com>
@anuragsoni
Copy link
Collaborator Author

@rgrinberg I think i've fixed my problems with travis (I was missing package field in the test stanza)
This should be ready for review now

@rgrinberg rgrinberg merged commit d7006c4 into rgrinberg:master Nov 29, 2018
@rgrinberg
Copy link
Owner

Thanks

rgrinberg added a commit to rgrinberg/opam-repository that referenced this pull request Jan 4, 2019
CHANGES:

- Switch to dune (rgrinberg/opium#88, anuragoni)

- Keep the "/" cookie default and expose all cookie directives (rgrinberg/opium#82,
  @actionshrimp)

- Do not assume base 64 encoding of cookies (rgrinberg/opium#74, @malthe)

- Add caching capabilities to middleware (rgrinberg/opium#76, @mattjbray)
@anuragsoni anuragsoni deleted the switch-to-opam-2 branch September 2, 2019 16:55
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.

None yet

2 participants