Hi ARIA TOSCA,

You have a new direct message from the ASF team
(https://the-asf.slack.com/x-185534614710-206058223746/).

---

@digestai
View in the archives:
https://the-asf.slack.com/x-185534614710-206058223746/archives/D5E4SCANM/p1498798845087266

Digest.AI (8:00 AM, June 30th)
Hi ARIA,
*Here’s your digest for June 30th 2017*
There are 60 messages yesterday, and the most active user is ran

*#ariatosca*
 @ran: :thumbsup: that's great Tal, Maxim and I have went over it and
besides a few tiny issues (mostly around some extras from rebase which
were no longer needed) I merged the PR
 We'll indeed need to talk about the right flow to make sure that the
docs are staying up to date and not only updated once per release
 weird that bad crossreferences don't raise any errors :confused:
 in any case im not sure if we have to review the resulting
documentation every PR, perhaps just building it (and making sure no
errors were raised) would be enough in most cases?
 In any case thank you for doing this, seems like you've put much effort
into it, so we'll definitely try not to mess it :wink:
 @ran: I've created a 0.1.0 package, placed it on `/dist/dev` and raised
a voting thread. Please see the relevant mail and vote
:slightly_smiling_face:
 @emblemparade: can we create a party coalition to override @johndament
? just kidding, we do need to fix those things :confused: the package
does install, but the instructions assume it's already on pypi, which is
actually wrong for a source distribution
 @ran: oh I totally get it, we should probably have tried pausing or
letting you merge first before introducing new changes as your changes
were pretty much all over the place :slightly_smiling_face: but then
again we didnt know how long it might take.
in any case the issues i remember were:
1) changed methods in modeling/relationships - maxim changed a couple of
methods there, and on your branch your kept both the old and new
versions.
2) renamed module `compilation.py` -- `graph_compilation.py` - on your
branch they both existed.

overall it wasn't so bad to find and fix these issues so
:slightly_smiling_face:



*Links shared by your team today:*



* * *

You can snooze these notifications for
an hour:
https://the-asf.slack.com/unsub/U5FFQJ2LW-5a1cb74d28-notify-mute-1h
eight hours:
https://the-asf.slack.com/unsub/U5FFQJ2LW-796e6535e1-notify-mute-8h
a day:
https://the-asf.slack.com/unsub/U5FFQJ2LW-4e6e073ada-notify-mute-1d
three days:
https://the-asf.slack.com/unsub/U5FFQJ2LW-806a2d8aa0-notify-mute-3d
or the next week:
https://the-asf.slack.com/unsub/U5FFQJ2LW-6adc840c34-notify-mute-7d.

You can also turn email notifications off:
https://the-asf.slack.com/unsub/U5FFQJ2LW-64de78982b-notify.

For more detailed preferences, see your account page:
https://the-asf.slack.com/account.

Reply via email to