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

Update elixir.yml #4

Merged
merged 1 commit into from
Oct 26, 2020
Merged

Update elixir.yml #4

merged 1 commit into from
Oct 26, 2020

Conversation

naomijub
Copy link
Contributor

Fixes failing PR #3

Fixes failing PR #3
This was referenced Oct 26, 2020
@psfblair psfblair merged commit 4735319 into psfblair:master Oct 26, 2020
@naomijub naomijub deleted the patch-1 branch October 26, 2020 17:09
@naomijub
Copy link
Contributor Author

Now we just need to publish the package and I forgot to update the README to use version 2.2.0.
Thanks

@psfblair
Copy link
Owner

Thank you for reminding me -- it's been so long since I've done any Elixir that I didn't even remember that I'd published this as a package. Let me figure out what I need to do.

@naomijub
Copy link
Contributor Author

It is easy, I have this Makefile to help me. I use make all and then make publish:

all: deps compile protocols

get-deps:
	rm -f mix.lock
	mix deps.get

deps: get-deps
	mix deps.compile

compile:
	mix compile

protocols:
	mix compile.protocols

clean-deps:
	mix deps.clean --all
	rm -rf deps

clean: clean-deps
	mix clean

test: compile
	mix test

docs:
	mix docs

lint: format
	mix credo --strict

format:
	mix format

outdated:
	mix hex.outdated

spec:
	 mix dialyzer --format dialyxir

publish:
	mix hex.publish
	mix hex.publish docs

@psfblair
Copy link
Owner

Thank you. I discovered a modification to the typespecs that had not been pushed, so I have created PR#5 with that change. I am going to merge that and then publish.

@psfblair
Copy link
Owner

Okay, the package is up - sorry about the delay; installing the latest Elixir turned out to be a bit of an ordeal.

@naomijub
Copy link
Contributor Author

Thanks!

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